manager-ui icon indicating copy to clipboard operation
manager-ui copied to clipboard

Content: Edit Content Item Page - Control P / Command P Shortcut to Publish Item does not work

Open zcolah opened this issue 1 year ago • 7 comments

Bug

The following two functions defined the spec of the edit content item page are not working. Please fix them.

  1. A user should be able to save and publish an item if they use Control P or Command P | Figma
  2. A user should be able to publish an item if they use Control P or Command P | Figma

Reference Articles

https://www.robin-drexler.com/2015/07/07/overriding-default-browser-shortcuts/ https://support.mozilla.org/kn/questions/1367908#:~:text=You%20can%20use%20the%20Browser,want%20to%20disable%20or%20override.

zcolah avatar Jan 29 '24 19:01 zcolah

@zcolah I tried Ctrl+P without changing so far in the code it brings a modal isn't this the expected behavior?

https://github.com/zesty-io/manager-ui/assets/44116036/e476fe9b-51f1-4a27-95da-1146509426eb

glespinosa avatar Feb 06 '24 10:02 glespinosa

@zcolah I think it's working fine also , I ask darwin to try it in their machine. Is cmd command working fine on your end?

glespinosa avatar Feb 12 '24 23:02 glespinosa

@glespinosa it is now working. Closing this issue.

zcolah avatar Feb 15 '24 04:02 zcolah

@glespinosa Command P is opening the print dialog. Can you look into why? CleanShot 2024-04-05 at 21 30 24

zcolah avatar Apr 05 '24 16:04 zcolah

@zcolah Upon checking I think because you're mouse is in focus on a textfield , I don't know if this issue existed previously. Try focusing it outside the textfield

glespinosa avatar Apr 08 '24 06:04 glespinosa

@glespinosa Could you then add an enhancement where the Command P and Control P shortcut works even if my house is focused on a text field, or is typing in an input?

zcolah avatar Apr 08 '24 11:04 zcolah

@zcolah upon checking again, while trying to reproduce the issue. It's now working fine.

https://github.com/zesty-io/manager-ui/assets/44116036/43e1e497-5e75-4f77-8486-c98b658ad6d6

glespinosa avatar Apr 08 '24 15:04 glespinosa