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

Add relationship-based options to status dropdowns #12377

Merged
merged 1 commit into from
Nov 19, 2019

Conversation

Gargron
Copy link
Member

@Gargron Gargron commented Nov 13, 2019

Instead of always showing mute/block, fetch up-to-date relationship when the status dropdown is opened, to be able to also show unmute/unblock as well as block/unblock the domain.

Move bookmark action in inline statuses from action bar to dropdown

@Gargron Gargron added the ui Front-end, design label Nov 13, 2019
@trwnh
Copy link
Member

trwnh commented Nov 14, 2019

fixes #10514 i think?

Move bookmark action in inline statuses from action bar to dropdown
@Gargron Gargron force-pushed the feature-dynamic-mute-options-in-dropdown branch from 8469564 to 7e935b2 Compare November 18, 2019 19:27
@Gargron Gargron requested a review from ykzts November 19, 2019 13:21
@Gargron Gargron merged commit 853a67e into master Nov 19, 2019
@Gargron Gargron deleted the feature-dynamic-mute-options-in-dropdown branch November 19, 2019 21:28
@umonaca
Copy link
Contributor

umonaca commented Nov 21, 2019

Well, not everyone wants the lovely bookmark icon to be removed...

@ClearlyClaire
Copy link
Contributor

ClearlyClaire commented Nov 24, 2019

Note that on non-mobile, this may cause the menu to change while displayed, because of the relationship info being updated after the menu gets shown.

On mobile, the behavior is different, with the updated relationship not updating the menu.

EDIT: Also, keyboard navigation gets reset if the current selected item gets updated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ui Front-end, design
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants