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

Merge tooltip strings for skins and controller wizard and maintain only one source #7743

Open
mixxxbot opened this issue Aug 22, 2022 · 5 comments

Comments

@mixxxbot
Copy link
Collaborator

Reported by: esbrandt
Date: 2014-12-18T06:37:41Z
Status: Confirmed
Importance: Medium
Launchpad Issue: lp1403748
Tags: polish, usability


Currently, we maintain 2 different string sources for tooltips:

  • src/skin/tooltips.cpp
  • src/controllers/controlpickermenu.cpp

Currently, we have the following quirks

  • Same control different content in the tooltip
  • Same control different spellings in the tooltip
  • Control missing tooltip in one or another source

This is unnecessary work for code and translation contributors as well.
We have manifests for effects including a description, what about something similar in a centralized fashion for control descriptions?

@mixxxbot
Copy link
Collaborator Author

Commented by: Be-ing
Date: 2017-11-11T15:49:55Z


Should we deal with this for 2.1 or postpone it?

@mixxxbot
Copy link
Collaborator Author

Commented by: daschuer
Date: 2017-11-12T09:11:16Z


It is OK for me to postpone it.

@mixxxbot
Copy link
Collaborator Author

Commented by: esbrandt
Date: 2017-11-12T14:20:59Z


Do do not give it another 3 years.

@mixxxbot
Copy link
Collaborator Author

Commented by: Be-ing
Date: 2017-11-12T15:23:59Z


jus, unless you are committing to take care of this for 2.2, I don't think it is helpful to assign bugs to a milestone. This is how milestones accumulate a lot of noise, which makes it hard to tell what actually needs to get done to make a release happen.

@mixxxbot
Copy link
Collaborator Author

Commented by: daschuer
Date: 2018-07-15T20:45:21Z


Related:
https://bugs.launchpad.net/mixxx/+bug/1781827

@mixxxbot mixxxbot transferred this issue from another repository Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant