Skip to content

Add-on Store: use Yes/No buttons with incompatibility question dialogs, instead of OK/Cancel buttons; improve related translator comments#15015

Merged
seanbudd merged 8 commits into
nvaccess:masterfrom
XLTechie:f15000
Jun 18, 2023
Merged

Conversation

@XLTechie

@XLTechie XLTechie commented Jun 16, 2023

Copy link
Copy Markdown
Collaborator

Link to issue number:

Fixes #15000

Summary of the issue:

Dialogs used when installing, enabling, or upgrading to, incompatible add-ons, asked yes or no questions to the user.
However, such dialogs used OK and Cancel as their buttons, which didn't really make sense.

Description of user facing changes

These three dialogs now have Yes and No buttons.

Incidentally, added a newline before the question in each dialog, as commented in the original issue. This can be reverted separately if undesirable.

Description of development approach

  • Replaced the ErrorAddonInstallDialogWithCancelButton class, with a new class called ErrorAddonInstallDialogWithYesNoButtons.
  • Removed ErrorAddonInstallDialogWithCancelButton, since it wasn't used anywhere other than for these three dialogs in gui/_addonStoreGui/controls/messageDialogs.py.
  • Added a newline to separate explanation from actual question in these dialogs.
  • Enhanced (hopefully) some translator comments I noticed along the way, as follow:
    • Created a more accurate translator comment for the incomparable version during update message.
    • Made the translator comment less ambiguous, for the message used when an add-on is being installed but is too old to be compatible.
    • Made the translator comment more accurate, for the message used when enabling an add-on too old to be compatible.
    • Fixed a typo in another translator comment.

Testing strategy:

  • Manually tested installing an incompatible add-on.
  • Manually tested enabling an incompatible add-on.

Haven't tested the case where an installed add-on version string is not able to be compared to a store version, but the dialog usage is the same as the other cases so it is likely to work as well as it did before this PR.

Known issues with pull request:

Even though the default button is set to "no" in these dialogs, "About add-on" is the focused default when their used.
This was true before as well with the "Cancel" button.
I'm not quite sure why SetDefault() is being ignored.

Change log entries:

The store is pre-release, none should be needed.

Code Review Checklist:

  • Pull Request description:
    • description is up to date
    • change log entries
  • Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • API is compatible with existing add-ons.
  • Documentation:
    • User Documentation
    • Developer / Technical Documentation
    • Context sensitive help for GUI changes
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English
  • Security precautions taken.

@XLTechie XLTechie marked this pull request as ready for review June 16, 2023 03:31
@XLTechie XLTechie requested a review from a team as a code owner June 16, 2023 03:31
@XLTechie XLTechie requested a review from seanbudd June 16, 2023 03:31

@seanbudd seanbudd left a comment

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.

looks good to me, other than the liblouis issue

Thanks @XLTechie

Comment thread source/gui/_addonStoreGui/controls/messageDialogs.py
@XLTechie

This comment was marked as resolved.

@seanbudd

Copy link
Copy Markdown
Member

Seems worse now - cldr has also changed

@XLTechie

This comment was marked as outdated.

@seanbudd

Copy link
Copy Markdown
Member

have you performed git submodule update --init --recursive && git submodule sync --recursive?

@XLTechie

This comment was marked as resolved.

@XLTechie XLTechie marked this pull request as draft June 16, 2023 05:51
@XLTechie XLTechie changed the title Add-on Store: use Yes/No buttons with incompatibility question dialogs, instead of OK/Cancel buttons Add-on Store: use Yes/No buttons with incompatibility question dialogs, instead of OK/Cancel buttons; improve related translator comments Jun 16, 2023
@XLTechie XLTechie requested a review from seanbudd June 16, 2023 06:49
@XLTechie

Copy link
Copy Markdown
Collaborator Author

@seanbudd the problem with submodules should be fixed here, sorry about that. I noticed various things I thought could be improved about the translator comments, and have suggested them since I was already in this module. See updated PR description or commits for details.

@seanbudd seanbudd marked this pull request as ready for review June 18, 2023 23:38

@seanbudd seanbudd left a comment

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.

Thanks @XLTechie

@seanbudd seanbudd merged commit 56763f0 into nvaccess:master Jun 18, 2023
@nvaccessAuto nvaccessAuto added this to the 2023.2 milestone Jun 18, 2023
@XLTechie XLTechie deleted the f15000 branch June 19, 2023 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add-on Store: Add-on incompatible / too old dialog: switch from OK/Cancel, to Yes/No

3 participants