Skip to content
This repository was archived by the owner on Sep 11, 2024. It is now read-only.

Disable jump to read receipt button instead of hiding when nothing to jump to - #12863

Merged
t3chguy merged 5 commits into
developfrom
t3chguy/fix/27883
Aug 6, 2024
Merged

Disable jump to read receipt button instead of hiding when nothing to jump to#12863
t3chguy merged 5 commits into
developfrom
t3chguy/fix/27883

Conversation

@t3chguy

@t3chguy t3chguy commented Aug 5, 2024

Copy link
Copy Markdown
Member

…en no RR

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy t3chguy added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements label Aug 5, 2024
@t3chguy t3chguy self-assigned this Aug 5, 2024
t3chguy added 3 commits August 5, 2024 15:20
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Comment thread src/i18n/strings/en_EN.json Outdated
"ignore_confirm_title": "Ignore %(user)s",
"invited_by": "Invited by %(sender)s",
"jump_to_rr_button": "Jump to read receipt",
"jump_to_rr_button_disabled": "This feature is disabled",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This feels a bit overly unhelpful. Conditional that shows either, "Not available in spaces" or, "No receipts for ${user}" depending on isSpace?

@t3chguy t3chguy Aug 6, 2024

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The tooltip is now gone (forgot to push), but this would have been as specified by design. See element-hq/element-web#27883 (comment)

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy
t3chguy added this pull request to the merge queue Aug 6, 2024
Merged via the queue into develop with commit 5519b81 Aug 6, 2024
@t3chguy
t3chguy deleted the t3chguy/fix/27883 branch August 6, 2024 15:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Jump to last RR has vanished

2 participants