maverick/docbrown
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Make sidebar billboard margins consistent and handle varied browser sizes for overflow (#19635)

Browse source
This commit is contained in:
Ben Halpern 2023-06-27 09:48:45 -04:00 • committed by GitHub
parent 62ff54cb94
commit 1ff5e91cce
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/assets/stylesheets/components/cards.scss
View file

@ -13,6 +13,8 @@
background: var(--card-bg);
color: var(--card-secondary-color);
box-shadow: 0 0 0 1px var(--card-secondary-border);
max-height: calc(100vh - var(--header-height) - 2*var(--layout-padding));
overflow: auto;
}
&--elevated {

4
app/views/articles/_sticky_nav.html.erb
View file

@ -1,5 +1,5 @@
<% @actor = @article.organization || @article.user %>
<div class="crayons-article-sticky grid gap-4 break-word" id="article-show-primary-sticky-nav">
<div class="crayons-article-sticky grid gap-4 pb-4 break-word" id="article-show-primary-sticky-nav">
<div class="crayons-card crayons-card--secondary branded-7 p-4 pt-0 gap-4 grid" style="border-top-color: <%= Color::CompareHex.new([user_colors(@actor)[:bg], user_colors(@actor)[:text]]).brightness(0.88) %>;">
<%= render "shared/profile_card_content", actor: @actor, context: "sidebar" %>
</div>
@ -55,4 +55,4 @@
<% end %>
</div>
<div class="crayons-article-sticky grid gap-4 break-word pt-4 js-display-ad-container" data-async-url="<%= article_display_ad_path(username: @article.username, slug: @article.slug, placement_area: :post_sidebar) %>"></div>
<div class="crayons-article-sticky grid gap-4 break-word js-display-ad-container" data-async-url="<%= article_display_ad_path(username: @article.username, slug: @article.slug, placement_area: :post_sidebar) %>"></div>