Skip to content

Added ability to sort add-ons by last tested version#18506

Merged
seanbudd merged 9 commits into
nvaccess:masterfrom
nvdaes:lastTestedVersion
Jul 23, 2025
Merged

Added ability to sort add-ons by last tested version#18506
seanbudd merged 9 commits into
nvaccess:masterfrom
nvdaes:lastTestedVersion

Conversation

@nvdaes

@nvdaes nvdaes commented Jul 18, 2025

Copy link
Copy Markdown
Collaborator
  • Add ability to sort add-ons by last tested NVDA version
  • Update changelog

Link to issue number:

Fixes #18440

Summary of the issue:

Sorting add-ons by last tested version can be helpful to know if an add-on is compatible with an updated version of NVDA, available but not installed yet. In this way, users can decide if they prefer to update NVDA, or wait until a compatible version of the add-on is available.

Description of user facing changes:

In add-on lists of the add-on store, a new column with last tested NVDA version is available, and add-ons can also be sorted by this column.

Description of developer facing changes:

None.

Description of development approach:

In addonStoreGui, viewModels, addonList module, a new field with its corresponding text has been added to create a new column, showing the last tested NVDA version

Testing strategy:

Open the store, switch to various tabs, check that a new column with last tested version is available, and that add-ons can be sorted by this column.

Known issues with pull request:

None.

Code Review Checklist:

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

@coderabbitai summary

@nvdaes

nvdaes commented Jul 19, 2025

Copy link
Copy Markdown
Collaborator Author

@seanbudd , po files contain errors, but I think that this is not related to this PR, so I'll mark this as ready for review.

@nvdaes nvdaes marked this pull request as ready for review July 19, 2025 13:42
@nvdaes nvdaes requested a review from a team as a code owner July 19, 2025 13:42
@nvdaes nvdaes requested a review from SaschaCowley July 19, 2025 13:42
@CyrilleB79

Copy link
Copy Markdown
Contributor

@nvdaes (author) or @SaschaCowley / @seanbudd (reviewers), what about the visual aspect of the list? Do all the columns fit in the list?

It would also be worth having the install date column in the Installed add-ons tab, and have add-ons sorted by install date. This would be very useful for trouble shouting, especially since automatic update is available.

I also wonder, at some point, if we should not give the capability to allow the user to choose which columns are displayed, as done in the detailed view of task manager, processes tab. And to be able to sort by any column.

All these additional proposals are probably off-topic with respect to this PR, but I mention theme here in case it may impact the choices done in this PR.

@nvdaes

nvdaes commented Jul 20, 2025

Copy link
Copy Markdown
Collaborator Author

Cyrille wrote:

Do all the columns fit in the list?

I'm totally blind and I cannot ensure it. But I think so, since, when I created the publication date column in other pull request, sometimes I noted that some columns weren't read, and this was fixed by changin column size.
I took this since this was not marked as sightedIdeal.

@seanbudd

Copy link
Copy Markdown
Member

@nvdaes - Can you also ensure the last tested and minimum API versions are always available in the add-on details panel? right now they are only visible in this installed add-ons tab.

)
lastTestedVersion = (
# Translators: The name of the column that contains the last version of NVDA tested with this add-on.
pgettext("addonStore", "Last tested NVDA version"),

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.

#18440 also requested adding minimum version

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.

can you also move these after date

@seanbudd

Copy link
Copy Markdown
Member

@CyrilleB79 - the text of the column header is going to be clipped for most resolutions, but the column is fine otherwise.

I agree that you should be able to sort by any column - visible or not, and choose what columns to display of all the available information an add-on can have. But this is probably better as a new issue

Comment thread source/gui/addonStoreGui/viewModels/addonList.py Outdated
@seanbudd seanbudd marked this pull request as draft July 21, 2025 00:17
@nvdaes nvdaes marked this pull request as ready for review July 21, 2025 19:33
@seanbudd seanbudd added the conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review. label Jul 22, 2025
Comment thread user_docs/en/changes.md Outdated
Co-authored-by: Sean Budd <seanbudd123@gmail.com>
@seanbudd seanbudd merged commit 1bdcdc9 into nvaccess:master Jul 23, 2025
18 of 20 checks passed
@SaschaCowley SaschaCowley added this to the 2025.3 milestone Aug 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add ability to quickly see and sort by minimum and last tested version of add-ons in the store

4 participants