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

Disable select when only one domain exiists #29873

Merged
merged 1 commit into from
Jan 3, 2019

Conversation

belcherj
Copy link
Contributor

@belcherj belcherj commented Jan 2, 2019

Changes proposed in this Pull Request

  • Disables select if only one option present
  • Hides down arrow if select is disabled

Testing instructions

  • Goto a site with a 1 custom domain name

  • Goto domains->email

  • Click either Add G Suite User or Add G Suite

  • Observe that the domain select is not a dropdown

  • Goto a site with a 2 custom domain names

  • Goto domains->email

  • Click either Add G Suite User or Add G Suite

  • Observe that the domain select is a dropdown

Before:
screen shot 2019-01-02 at 4 41 00 pm

After:
screen shot 2019-01-02 at 4 22 47 pm

@belcherj belcherj added the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Jan 2, 2019
@belcherj belcherj self-assigned this Jan 2, 2019
@belcherj belcherj requested a review from a team January 2, 2019 21:41
@matticbot
Copy link
Contributor

@stephanethomas
Copy link
Contributor

Changes look good, and work fine.

@stephanethomas stephanethomas added [Status] Ready to Merge G Suite and removed [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. labels Jan 3, 2019
@belcherj belcherj merged commit f890d85 into master Jan 3, 2019
@belcherj belcherj deleted the fix/disable-select-with-single-option branch January 3, 2019 15:06
blowery added a commit that referenced this pull request Jan 3, 2019
…sh-2019

* origin/master:
  Disable select when only one domain exiists (#29873)
  Tiled galleries: Disable captions (#29776)
  Prevent scrolling up when opening a dialog (#29832)
  Gutenberg: Move Shortlinks to production (#29883)
  Gutenlypso: add convert to blocks dialog (#29790)
  Gutenberg: Move Related Posts to production blocks (#29650)
  Analytics: fix ad-tracking issue on signup for non-gdpr countries (#29741)
  Update mobile phone validation module (used for 2fa) (#29740)
  Gutenlypso: make sure titles load on second edit (#29877)
  Site Picker: Change wording of /page and /block-editor to match /post (#29859)
  Gutenberg: update copy link in page list to be editor aware
  Gutenberg: use core approach of initialEdits over overridePost
  Gutenberg: when duplicating a post, override post content
  Gutenberg: update duplicate url when Gutenlypso is enabled
  Refactor: Replace use of key-mirror with inline code (#29857)
  Fixes wrong selected domain name (#29824)
  Turn off prettier for SASS, use stylelint instead (#29697)
  Gutenberg: Add copy button to Shortlinks (#29556)
  add a section name to the body class (#29563)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants