-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Add allowOverridingNativeContextMenus()
#17777
Conversation
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
This requires matrix-org/matrix-react-sdk#5672 but that one requires this. Is there a safe order to merge these? |
No, the only way is to press both buttons at the same time :P |
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This "half" looks sane!
No longer the case, required bits landed as part of matrix-org/matrix-react-sdk#7850 also |
Requires: matrix-org/matrix-react-sdk#5672
element-web notes: none
This change is marked as an internal change (Task), so will not be included in the changelog.