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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Following message id anchor hides message under topbar #9

Closed
JC3 opened this issue Jun 8, 2017 · 2 comments
Closed

Following message id anchor hides message under topbar #9

JC3 opened this issue Jun 8, 2017 · 2 comments
Assignees
@JC3
Labels
bug
Projects
ChatTopBar

Comments

@JC3
Copy link
Owner

JC3 commented Jun 8, 2017 •
edited

Need to offset anchor pos somehow.

http://chat.meta.stackexchange.com/transcript/message/6091755#6091755
@JC3 JC3 added the bug label Jun 8, 2017
@JC3 JC3 self-assigned this Jun 8, 2017
@JC3 JC3 added this to Issues in ChatTopBar Jun 8, 2017
@JC3 JC3 moved this from Issues to Finished in ChatTopBar Jun 9, 2017
@JC3 JC3 moved this from Finished to Issues in ChatTopBar Jun 9, 2017
@JC3
Copy link
Owner Author

JC3 commented Jun 9, 2017

fixed in dev but note to self: make sure scrolling from ping notification counter works correctly too. need to implement and test.

@JC3
Copy link
Owner Author

JC3 commented Jun 9, 2017

unread notification counter seems to work correctly. will keep an eye out.

fixed in 672594a

@JC3 JC3 closed this as completed Jun 9, 2017
@JC3 JC3 moved this from Issues to Finished in ChatTopBar Jun 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JC3 JC3

Labels
bug
Projects
ChatTopBar
Finished
Milestone
No milestone
Development

No branches or pull requests
1 participant
@JC3