AllBugsFeaturesChangelogSearchNew post
Sign in with Fluxer
Bugs

Timestamps in dense mode are hidden

(#270) Bug Awaiting confirmation ui
by RexSystem edited originally by @suspiciouslyBee on GitHub

Summary

In dense mode, the timestamps to the left of each message are hidden. I did some digging and figured out that Fluxer does indeed calculate the time and put it in the correct spot, but the overflow culling cuts it out since it is outside of the message container bounds. When I manually force the generated compactContentWrapper message element to show all overflow in the Message.module.css file using a browser inspector, the stamp is visible and fully functional.

Steps to reproduce

  1. Set mode to dense
  2. Observe how messages do not have timestamps that would otherwise have them in cozy mode.

4 comments

Sign in with Fluxer to comment and vote.
Comment by Rex
RexSystem 1 vote
Status changed from Fixed to Awaiting confirmation
This was closed in a bulk cleanup before Fluxer V2 without being checked or fixed. It may work now, so it is waiting for someone to confirm whether the bug still happens.
Permalink
Comment by @suspiciouslyBee
RexSystem 1 vote originally by @suspiciouslyBee on GitHub OP
I may have a fix, doing local testing before submitting a PR. Might be a regression with the refactor branch specifically. Edit: I'm tempted to just put the change up without offline local testing because I can't get the client to run and its literally changing two lines of CSS
Permalink
Comment by @suspiciouslyBee
RexSystem 1 vote originally by @suspiciouslyBee on GitHub OP
It seems that PRs are currently disabled so I will come back to tidy/test this later
Permalink
Comment by @VixusFoxy
RexSystem 1 vote originally by @VixusFoxy on GitHub
Confirming: Timestamps are not shown in the Dense display setting under Look & Feel.
Permalink