Skip to content

gtk: add "remember choice" toggle for clipboard confirmation dialog #6783

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

pluiedev
Copy link
Member

Implements #6763 (not backporting to the 1.2 version since I'm way too lazy)

image

@pluiedev pluiedev requested a review from a team as a code owner March 17, 2025 21:53
Copy link
Member

@jcollie jcollie left a comment

Choose a reason for hiding this comment

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

I dont't think that we need to make this work on Debian 12, but it should at least compile ;)

@pluiedev pluiedev force-pushed the push-vwyvuxqlnmyu branch from afb1fe5 to 5a574d3 Compare March 18, 2025 07:34
@pluiedev pluiedev requested a review from jcollie March 18, 2025 08:13
Fuzzy matched strings are really annoying to pick out and remove for
later. We should just leave strings untranslated until localizers
get around to them, instead of letting gettext's (fairly naïve) algorithm
to do that for us.

No translation is better than a bad guesswork translation.
@pluiedev pluiedev force-pushed the push-vwyvuxqlnmyu branch from 5a574d3 to ea6f216 Compare March 18, 2025 11:38
@tristan957
Copy link
Member

Should the text say something about the choice being remembered for the session?

@tristan957
Copy link
Member

I also left a comment at #6763 (comment)

@pluiedev
Copy link
Member Author

The problem is that "session" is a very nebulous term, even more so in other languages (trying to express it in Chinese actually almost drove me insane). I think just implying that it only has an effect for this split, and that it will be reset after reloading the config, has pretty much the same effect.

@pluiedev
Copy link
Member Author

Implementing it per surface is very easy and I think could serve as an initial implementation. Per process would be a lot more complicated but perhaps still doable — out of scope for this PR though IMO

@tristan957
Copy link
Member

tristan957 commented Mar 18, 2025

I guess I'm confused about what session even means. Is it per ghostty instance or per surface? Referring to the original discussion

@jcollie
Copy link
Member

jcollie commented Mar 18, 2025

I guess I'm confused about what session even means. Is it per ghostty instance or per surface? Referring to the original discussion

The current PR is per-surface instance correct? So 5 tabs with 2 splits each you could potentially be asked 10 times. So anytime you create a new surface you'd be asked again. Also, I believe that if you reload your config you'd reset all that to default so you'd be asked again, even for surfaces where you'd already been asked.

Mainly we just need to be clear with the documentation. Maybe there's a way to add a little "?" button that would explain in more detail than would fit in the confirmation dialog.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants