Skip to content

chore: remove dialog showHiddenFiles support on Linux#51704

Merged
jkleinsc merged 2 commits into
mainfrom
chore/remove-show-hidden-files-file-dialog-support-on-Linux
Jun 4, 2026
Merged

chore: remove dialog showHiddenFiles support on Linux#51704
jkleinsc merged 2 commits into
mainfrom
chore/remove-show-hidden-files-file-dialog-support-on-Linux

Conversation

@ckerr

@ckerr ckerr commented May 19, 2026

Copy link
Copy Markdown
Member

Description of Change

Fixes #46817.

Remove showHiddenFiles support from the dialog API on Linux.

This was deprecated in b9a09ac / #46926 for 42-x-y. This PR removes it.

Checklist

Release Notes

Notes: Removed showHiddenFiles support from the dialog API on Linux.

@ckerr ckerr requested a review from a team as a code owner May 19, 2026 20:06
@ckerr ckerr added platform/linux semver/major incompatible API changes target/43-x-y PR should also be added to the "43-x-y" branch. labels May 19, 2026
@ckerr ckerr marked this pull request as draft May 19, 2026 20:13
@ckerr ckerr marked this pull request as ready for review May 20, 2026 14:33
@ckerr ckerr added semver/none semver/major incompatible API changes and removed semver/major incompatible API changes api-review/requested 🗳 semver/none labels May 21, 2026

@samuelmaddock samuelmaddock 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.

API LGTM

Comment thread docs/breaking-changes.md
When the cookie being set is identical to an existing one (same name, domain, path, and value, with no actual changes), the change cause is `inserted-no-change-overwrite`.
When the value of the cookie being set remains unchanged but some of its attributes are updated, such as the expiration attribute, the change cause will be `inserted-no-value-change-overwrite`.

### Deprecated: `showHiddenFiles` in Dialogs on Linux

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.

I think we should keep this heading for historical purposes.

@jkleinsc jkleinsc 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.

@ckerr we shouldn't remove the deprecation from the breaking changes as @erickzhao suggests.

@electron-cation electron-cation Bot removed the new-pr 🌱 PR opened recently label May 27, 2026
ckerr added 2 commits June 3, 2026 12:08
This was deprecated in Feb by b9a09ac for 42-x-y.
This PR removes it for main + backport to 43-x-y.

Xref: #46817
@ckerr ckerr force-pushed the chore/remove-show-hidden-files-file-dialog-support-on-Linux branch from 8639a8f to 368ee26 Compare June 3, 2026 17:10
@ckerr

ckerr commented Jun 3, 2026

Copy link
Copy Markdown
Member Author

Rebased to main + the deprecation announcement reinstated in 368ee26

@ckerr ckerr requested review from erickzhao and jkleinsc June 3, 2026 17:10

@erickzhao erickzhao 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.

API LGTM

@samuelmaddock samuelmaddock 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.

API LGTM

@jkleinsc jkleinsc merged commit b320f91 into main Jun 4, 2026
76 checks passed
@jkleinsc jkleinsc deleted the chore/remove-show-hidden-files-file-dialog-support-on-Linux branch June 4, 2026 17:02
@release-clerk

release-clerk Bot commented Jun 4, 2026

Copy link
Copy Markdown

Release Notes Persisted

Removed showHiddenFiles support from the dialog API on Linux.

@trop

trop Bot commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

I have automatically backported this PR to "43-x-y", please check out #51880

@trop trop Bot added in-flight/43-x-y merged/43-x-y PR was merged to the "43-x-y" branch. and removed target/43-x-y PR should also be added to the "43-x-y" branch. in-flight/43-x-y labels Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api-review/approved ✅ merged/43-x-y PR was merged to the "43-x-y" branch. platform/linux semver/major incompatible API changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GTK3 file chooser overwrites "Show hidden files" setting in Linux/GNOME

4 participants