Page MenuHomeWildfire Games

Do not display right click tooltip when it is not supported right now
AbandonedPublic

Authored by Silier on Sep 6 2020, 4:40 PM.

Details

Reviewers
None
Summary

disable tooltip to right click on buttons when it is not supported to not cause confusion for users

Test Plan

Check tooltip is displayed when right click action is possible, else it is hidden.

Diff Detail

Repository
rP 0 A.D. Public Repository
Branch
/ps/trunk
Lint
Lint OK
SeverityLocationCodeMessage
Warningbinaries/data/mods/public/gui/session/selection_panels.js:50ESLintBear (default-case)ESLintBear (default-case)
Warningbinaries/data/mods/public/gui/session/selection_panels.js:61ESLintBear (default-case)ESLintBear (default-case)
Warningbinaries/data/mods/public/gui/session/selection_panels.js:302ESLintBear (no-multi-spaces)ESLintBear (no-multi-spaces)
Warningbinaries/data/mods/public/gui/session/selection_panels.js:729ESLintBear (default-case)ESLintBear (default-case)
Warningbinaries/data/mods/public/gui/session/selection_panels.js:763ESLintBear (space-before-function-paren)ESLintBear (space-before-function-paren)
Unit
No Unit Test Coverage
Build Status
Buildable 13133
Build 26091: Vulcan BuildJenkins
Build 26090: Vulcan Build (macOS)Jenkins
Build 26089: Vulcan Build (Windows)Jenkins
Build 26088: arc lint + arc unit

Event Timeline

Silier created this revision.Sep 6 2020, 4:40 PM
Owners added a subscriber: Restricted Owners Package.Sep 6 2020, 4:40 PM
Silier requested review of this revision.Sep 6 2020, 4:45 PM
bb added a subscriber: bb.Sep 6 2020, 9:05 PM

Don't feel this is the correct way of approaching the issue. I rather have the buttons enabled for the right click.

Silier added a comment.Sep 7 2020, 8:29 AM

sure, if that diff makes it to a24, it would be great

Krinkle added a subscriber: Krinkle.