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

xfd: Adjust portlet tooltip to be more accurate #1193

Merged
merged 2 commits into from
Dec 12, 2020

Conversation

Amorymeltzer
Copy link
Collaborator

This was briefly discussed on WT:TW, but the portlet tooltip of Start a deletion discussion is rarely, if ever, sufficiently accurate, and indeed elides and even hides functionality (namely, RM). I've put the two ideas up here:

  • 1st commit: Start a discussion
  • 2nd commit: Be more specific for each venue (suggested by @siddharthvp, correct me if I misinterpreted)

I (clearly) favor the former as being simpler and not duplicating checks, but honestly the more detail from the second is probably nicer and likely to be more popular. It'll be inaccurate if folks choose something else (or in the specific cases of things like userboxes, stubs, etc.) but by the time the module window is open, I don't think anyone cares what the tooltip was.

@Amorymeltzer Amorymeltzer changed the title xfd: Adjust portlet tooltip to be more accurate (WIP) xfd: Adjust portlet tooltip to be more accurate Nov 16, 2020
@Amorymeltzer Amorymeltzer marked this pull request as draft November 16, 2020 13:29
@Amorymeltzer Amorymeltzer marked this pull request as ready for review November 30, 2020 21:05
@Amorymeltzer Amorymeltzer added this to the December 2020 update milestone Dec 7, 2020
@Amorymeltzer Amorymeltzer merged commit 6564548 into wikimedia-gadgets:master Dec 12, 2020
@Amorymeltzer Amorymeltzer deleted the xfd-tooltip branch December 12, 2020 22:00
@Amorymeltzer
Copy link
Collaborator Author

Went option 2

@siddharthvp siddharthvp mentioned this pull request Dec 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant