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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Timestamps Age Update #1694

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

motackt
Copy link

@motackt motackt commented Apr 27, 2019

Take 2! (from #1655)

I was surprised to see that work on timestamps went ahead so quick, but I hope these features can be added to make it even better!

New Features:

  • Users can display the timestamp next to the header
  • Timestamps now display how old they are!

look_at_that_padding_work_dayum

Known bugs:

  • Peepr post headers have a maximum width that cuts off long timestamps (ie with both relative and absolute time displayed); can fix by changing CSS for peepr post headers, but seems outside scope of this extension?

(Also, I have no idea why that first commit is there, hopefully that's not a problem?)
(Also also, idk why the last edit in timestamps.js is there either bc i didn't make that edit?)

@motackt
Copy link
Author

motackt commented Apr 28, 2019

Considering having timestamps that are beside the header float to the right so it aligns with the reblog timestamps, i think it might look neater. Thoughts :?

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.

1 participant