Skip to content

Add 'Show hidden' check to file selector#5269

Merged
dacap merged 1 commit into
aseprite:mainfrom
Liebranca:show-hidden-files
Jul 18, 2025
Merged

Add 'Show hidden' check to file selector#5269
dacap merged 1 commit into
aseprite:mainfrom
Liebranca:show-hidden-files

Conversation

@Liebranca

Copy link
Copy Markdown
Contributor

Adds a checkbox to control showing hidden files in file selector window. Fix #3079.

@Liebranca Liebranca requested a review from dacap as a code owner July 11, 2025 02:46

@aseprite-bot aseprite-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clang-tidy made some suggestions

Comment thread src/app/ui/file_list.h
double zoom() const { return m_zoom; }
void setZoom(const double zoom);
void animateToZoom(const double zoom);
void setShowHidden(const bool show);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

warning: parameter 'show' is const-qualified in the function declaration; const-qualification of parameters only has an effect in function definitions [readability-avoid-const-params-in-decls]

Suggested change
void setShowHidden(const bool show);
void setShowHidden(bool show);

@dacap

dacap commented Jul 18, 2025

Copy link
Copy Markdown
Member

It works great @Liebranca 👍 I'll merge it as it is and then fix some issue on Windows to handle special hidden files.

Later we'll think in an alternative icon for this button, but I'm not sure what to use yet.

@dacap dacap merged commit bbab4d5 into aseprite:main Jul 18, 2025
12 checks passed
@dacap dacap added this to the v1.3.15 milestone Jul 18, 2025
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.

Hard to navigate to dot-prefixed folders in file dialogs

3 participants