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

Right-click menu not working in Full-screen mode #263

Open
SusanBrown opened this issue May 18, 2020 · 5 comments
Open

Right-click menu not working in Full-screen mode #263

SusanBrown opened this issue May 18, 2020 · 5 comments

Comments

@SusanBrown
Copy link
Contributor

The right-click menu is not working in Full-screen mode, either in CWRC or the Git-writer

If you right click on an entity or a tag in either the Tag menu or the editing panel you should get a popup menu

@lucaju
Copy link
Contributor

lucaju commented May 20, 2020

I found this on Stackoverflow: https://stackoverflow.com/questions/18578244/displaying-elements-other-than-fullscreen-element-html5-fullscreen-api

The context menu is created as a sibling to the main CWRC-Writer container (div). When activating 'fullscreen", the browser assigns the maximum z-index [2147483647] to this container (nothing can get in from it), not even when we try to override the browser using the !important keyword. That is the reason the context menu doesn't 'pop' on fullscreen mode.

A possible solution is to move the context-menu DIV to inside the main CWRC-Writer DIV. I manually did this in the browser and it seems to work.

@ilovan, should I spend time with it, or should we wait until I get to redesign the context-menu?

@SusanBrown
Copy link
Contributor Author

This is very encouraging, Luciano. Great to see it and that you think it is a solution. Presuming that the high priority tags here are up to date and still pending, I think that for now we should focus on the other high-priority issues, given that an update of CWRC-Writer needs to be fit in with the next update of CWRC which will be as soon as possible, and disable the full-screen button in the editor for that release. Once the other more urgent issues re: the schema problems have been dealt with, then spending some time on this if the update is not ready to go, makes sense to me. hope that's clear!

@lucaju
Copy link
Contributor

lucaju commented May 21, 2020

Ok.
I'm in the end tail to push code to that solves some problems with mapping schema.
Already disabled full-screen button until we fix this issue.
I will check the code on context-menu to see how much work I have to put on to solve this.

@lucaju
Copy link
Contributor

lucaju commented May 26, 2020

Done: 23376ab

@lucaju
Copy link
Contributor

lucaju commented Jun 3, 2020

The context menu is slightly displaced from where the user click. This only happens in the integrated version.

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

No branches or pull requests

3 participants