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

Ben/article show redesign (#245)

Browse source 
* Add sidebar sticky elements

* Add sticky sidebar to article show page

* Add saved changes check to user

* Fix missing CSS variable in sticky nav file

* Add date joined as default summary

* Remove accidentally-included file

* Add check for published on article sidebar

* Fix styling issues on side nav

* Modify article order

* Fix issue in side bar

* Fix issue in side bar
This commit is contained in:
Ben Halpern 2018-04-25 18:36:59 -04:00 • committed by GitHub
parent 7ce70f5333
commit ddf6ad45e1
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
app/assets/stylesheets/sticky-nav.scss
View file

@ -24,7 +24,6 @@
border-radius: 360px;
margin-right: 5px;
vertical-align: -9px;
}
.primary-sticky-nav-author-name{
font-size:1.3em;

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

@ -40,10 +40,10 @@
</div>
<% Article.tagged_with((@article.cached_tag_list_array + ["career","productivity","discuss"]), any: true).
includes(:user).
where("positive_reactions_count > ? OR comments_count > ?", (Rails.env.production? ? 17 : -1 ), (Rails.env.production? ? 7 : -1 )).
where("positive_reactions_count > ? OR comments_count > ?", (Rails.env.production? ? 20 : -1 ), (Rails.env.production? ? 8 : -1 )).
where(published: true).
where("featured_number > ?", 1449999999).
order("published_at DESC").
where("featured_number > ?", 5.days.ago.to_i).
order("RANDOM()").
limit(8).
each do |article| %>
<a class="primary-sticky-nav-element" href="<%= article.path %>">