Skip to content

Add EXIF columns to Library in List Mode (ISO, aperture, shutter, focal length) with sorting#961

Open
savukadoo wants to merge 1 commit intoCyberTimon:mainfrom
savukadoo:feature/library-exif-columns
Open

Add EXIF columns to Library in List Mode (ISO, aperture, shutter, focal length) with sorting#961
savukadoo wants to merge 1 commit intoCyberTimon:mainfrom
savukadoo:feature/library-exif-columns

Conversation

@savukadoo
Copy link
Copy Markdown

…and persistence

Description

Adds dynamic EXIF columns to the Library list view with sorting and persistence across navigation.

Type of Change

  • New feature
  • UI/UX improvement

Changes Made

  • Added selectable EXIF columns: ISO, Aperture, Shutter Speed, Focal Length
  • Enabled sorting on all EXIF columns
  • Persisted selected columns when switching between Library and Editor
  • Fixed EXIF field mapping (Aperture ↔ Shutter)
  • Standardized focal length display (mm)
  • Implemented numeric shutter speed sorting (handles fractional values correctly)
  • Fixed column/data alignment issues in list view

Screenshots/Videos

Screenshot 2026-04-03 at 18 30 23 Screenshot 2026-04-03 at 18 30 36

Testing

  • I have tested these changes locally and confirmed that they work as expected without issues

Test Configuration:

  • OS: macOS Tahoe
  • Hardware: Apple M2

Checklist

  • My code follows the project's code style
  • I haven't added unnecessary AI-generated code comments
  • My changes generate no new warnings or errors

Additional Notes

AI Disclaimer:

Please state the involvement of AI in this PR:

  • This PR is AI-generated but guided by a human

@savukadoo savukadoo requested a review from CyberTimon as a code owner April 3, 2026 08:31
@CyberTimon
Copy link
Copy Markdown
Owner

@savukadoo
Thanks for your PR. In normal thumbnail view, I'm now also able to multi select EXIF sort keys. This seems buggy, what do you think?

@savukadoo
Copy link
Copy Markdown
Author

savukadoo commented Apr 7, 2026 via email

@savukadoo
Copy link
Copy Markdown
Author

I just had a play and I assume your question is today without this change you can use only one sort key and now with this change you can use multiple sort keys. I did some testing and you can remove and add and go back to one key, its sort of flexible in thumbnail view like it was before and now just has an extra dimension.

I have been thinking about PR's for RR a little and as you are probably going to see now and into the future a tsunami of them coming in for review as this app gains momentum and AI tech is evolving fast. So, is there a better way that the community could vote? and I am only focusing this statement on major functionality that would effect workflows and major changes to the app. You ultimately or a small steering committee decide what goes up for vote, however that way the users can decide what they think is useful and beneficial.

I personally like this PR change, however its an opinion on my workflow and how it will benefit me, however like you , you may have a different perspective and that's a good thing as well.

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.

2 participants