pixel-paint icon indicating copy to clipboard operation
pixel-paint copied to clipboard

Consolidate all the shortcuts #43

Open Morka951 opened this issue 7 months ago • 6 comments

The Issues #43

Remove the "How to Copy Selection" blurb entirely from the app
Add "Click Drag" to the Shortcut instructions, courtesy of @nellyvo's recently merged PR
Add all shortcuts

Morka951 avatar May 25 '25 03:05 Morka951

Hey , Do you want the display to look like this? I haven’t added the blur effect yet — I’m just waiting to make sure I understood what you wanted first. Is this how you want the display to look? Capture d’écran du 2025-05-27 01-22-26

Morka951 avatar May 26 '25 23:05 Morka951

Hey , Do you want the display to look like this? I haven’t added the blur effect yet — I’m just waiting to make sure I understood what you wanted first. Is this how you want the display to look?

Hey @Morka951 sorry for the late response! (juggling a work deadline).

This looks awesome, yes this is what I meant!

I have some feedback that I wanted to share (removing outline of the modal, reordering the list in a more logical way) but honestly it is not a blocker at all. The blur I would say is not a blocker from merging. :)

Whenever you have the modal feature in place, feel free to ping me and I will try it out and review.

Kully avatar May 29 '25 23:05 Kully

Hey @Kully ,

I reordering the list in a more logical way.

For The blur i tried but this doesnt work i will try later . i thinks i fails my pr . Can you tell me if is it good because I had one commit behind on my branch.

Morka951 avatar May 30 '25 01:05 Morka951

i thinks i fails my pr . Can you tell me if is it good because I had one commit behind on my branch.

What do you mean by this? I am confused.

Are you saying that you finished the PR?

Kully avatar May 30 '25 02:05 Kully

@Kully Yes I finish the pr but when I push they told me conflict merge in the css . Overwrite I thinks , can you see if it's good ?

Morka951 avatar May 30 '25 09:05 Morka951

@Kully Yes I finish the pr but when I push they told me conflict merge in the css . Overwrite I thinks , can you see if it's good ?

@Morka951 Hmm, this just sounds like a git flow thing. I would look up "resolving git conflict" or something like that on YouTube, maybe for your IDE of choice.

If you don't find what you are looking for, feel free to drop a screenshot and I will take a look. :)

Kully avatar Jun 01 '25 19:06 Kully

Closing this PR in favour #52

Kully avatar Jun 28 '25 13:06 Kully