Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Article Actions] Using position sticky #945

Conversation

juanmanuelramallo
Copy link
Contributor

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Documentation Update

Description

Changed position of article actions bar (or reactions).
Keep bar in the bottom of the article section using position: sticky.
This was recently discussed here.
Main motivation on this is that it doesn't make sense (at least to me 馃檲) to have reaction buttons for the article when you are below its container, cause you are actually seeing suggestions to read other articles.

Related Tickets & Documents

none

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

Gif showing it in action
Reactions bar

Added to documentation?

  • docs.dev.to
  • readme
  • no documentation needed

[optional] What gif best describes this PR or how it makes you feel?

https://imgur.com/gallery/aX1c9FW

Copy link
Contributor

@benhalpern benhalpern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is so beautiful

@benhalpern benhalpern merged commit 6ada731 into forem:master Oct 18, 2018
@juanmanuelramallo juanmanuelramallo deleted the feature/position-sticky-to-reaction-bar branch October 18, 2018 18:32
@benhalpern
Copy link
Contributor

Hey @juanmanuelramallo we actually had to revert this change.

There were some weird small issues with hovering and 1px jumps.

(Plus the removal of overflow: hidden caused some issues but we could overcome that if the other things were non-issues.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants