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

GH-1244 add setting -webkit-user-select and others for MenuWrapper #1292

Merged
merged 3 commits into from
Sep 21, 2021

Conversation

sbishel
Copy link
Collaborator

@sbishel sbishel commented Sep 20, 2021

Summary

Adds settings
-webkit-user-select: text; /* Chrome all / Safari all /
-moz-user-select: text; /
Firefox all /
-ms-user-select: text; /
IE 10+ */

for .MenuWrapper *:firstChild{}

All other places we set user-select: text;, we set these as well.

Related to -
#1094

Ticket Link

Fixes #1244

@sbishel sbishel requested a review from a team as a code owner September 20, 2021 21:06
@sbishel sbishel requested review from ogi-m, asaadmahmood and jespino and removed request for a team September 20, 2021 21:06
@sbishel sbishel added 2: Dev Review Requires review by a core committer 2: QA Review Requires review by a QA tester labels Sep 20, 2021
@sbishel sbishel added this to the v0.9 milestone Sep 20, 2021
@sbishel sbishel added the CherryPick/Candidate A candidate for a quality or patch release, but not yet approved label Sep 20, 2021
Copy link
Contributor

@jespino jespino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sbishel sbishel removed the 2: Dev Review Requires review by a core committer label Sep 21, 2021
Copy link
Contributor

@ogi-m ogi-m left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and passed

  • Verified it's working in Safari
  • Tested in Chrome and Firefox

@ogi-m ogi-m removed the 2: QA Review Requires review by a QA tester label Sep 21, 2021
@sbishel sbishel added 3: Reviews Complete All reviewers have approved the pull request CherryPick/Approved Meant for the quality or patch release tracked in the milestone and removed CherryPick/Candidate A candidate for a quality or patch release, but not yet approved labels Sep 21, 2021
@sbishel sbishel merged commit 660fa5e into main Sep 21, 2021
@sbishel sbishel deleted the gh-1244-safari branch September 21, 2021 16:25
@mattermod
Copy link
Contributor

Cherry pick is scheduled.

@mattermod
Copy link
Contributor

Error trying doing the automated Cherry picking. Please do this manually

fatal: No such remote 'upstream'

+++ Returning you to the main branch and cleaning up.

sbishel added a commit that referenced this pull request Sep 21, 2021
…1292)

* add setting -webkit-user-select and others for MenuWrapper

* fix lint, disable check
@sbishel sbishel added the CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone label Sep 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3: Reviews Complete All reviewers have approved the pull request CherryPick/Approved Meant for the quality or patch release tracked in the milestone CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Can't enter more than one character in property names with Safari
4 participants