Skip to content
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

reset colors #1068

Closed
wants to merge 2 commits into from
Closed

reset colors #1068

wants to merge 2 commits into from

Conversation

Doublonmousse
Copy link
Contributor

To fix #1056

A few things I'm not too sure about

  • Is the position of the button correct ?

image

  • should other color things (that are in the document settings like format border color, pattern color and background color) be affected ?

- position of the button correct ?
- should other color things (in the document settings) be reset ?
@flxzt
Copy link
Owner

flxzt commented Apr 12, 2024

Shouldn't this be a reset for all color picker palette colors as described in issue #1056 ?

@Doublonmousse
Copy link
Contributor Author

I'm not 100 % sure. Given the current position, one would expect it only applies to color picker palettes of the Document part of settings.
But I don't want to mix things that affect the document (document background, grid color) with ones that don't (choice of pen colors).

Ideally this button should be in the color picker. But that means modifying the color picker (where almost all default colors are presented, though not in the exact same order)
image

The current position of the button is very similar to the revert/apply buttons for the page format. Maybe It'd be better to mirror this for the Document section and have a reset/templates button combo (with the templates part to populate maybe later)

@Doublonmousse
Copy link
Contributor Author

Maybe it'd be better to add a right click menu (that would also be triggered by a long touch press, and maybe pen long press) for each color in the color picker window to reset the colors.

(similar to how a right click in the color picker shows a customize option)

That could also be done for the individual color setters that are in the settings panel.

@Doublonmousse
Copy link
Contributor Author

Well I tried to detect right clicks and long press gestures on each RnColorSetter but only right click gesture detection works. Long press will only work once for each color setter then stop working.

Not sure how to do it correctly and where to put code between RnColorSetter and RnColorPicker, so if anyone has a better idea on how to do that (I would've added a dropdown menu with at least a reset option for each color), feel free to continue this PR.

Will close it otherwise

@flxzt
Copy link
Owner

flxzt commented Apr 25, 2024

The long-press Gtk gesture has some issues, it's finicky to make it work. I've had similar issues with it in the past. I think the idea to implement the reset in this is very good in general, maybe we can revisit it at some point..

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.

Option to reset the colors
2 participants