Skip to content

Conversation

@jeherve
Copy link
Member

@jeherve jeherve commented Mar 9, 2018

Fixes #5962 again.

When I worked on #7296 to first fix this issue, I wanted to change from the old unresponsive dropdown only layout (2)
to the new vertical layout that would be responsive (0).
But while addressing other feedback on the PR I reverted that change, ant the old layout remained the default.

This fixes that. The default is 0 (vertical layout) now.

Related WordPress.com diff: D10767-code

Testing instructions:

  1. On a site using the Google Translate Widget, apply the patch.
  2. The widget should go from

screen shot 2018-03-09 at 12 53 17

To

screen shot 2018-03-09 at 12 43 28

Proposed changelog entry for your changes:

  • Google Translate Widget: make sure the widget is responsive by default.

Fixes #5962 again.

When I worked on #7296 to first fix this issue, I wanted to change from the old unresponsive dropdown only layout (`2`)
to the new vertical layout that would be responsive (`0`).
But while addressing other feedback on the PR I reverted that change, ant the old layout remained the default.

This fixes that. The default is 0 (vertical layout) now.
@jeherve jeherve added [Type] Bug When a feature is broken and / or not performing as intended [Feature] Extra Sidebar Widgets [Status] Needs Review This PR is ready for review. [Pri] Normal labels Mar 9, 2018
@jeherve jeherve self-assigned this Mar 9, 2018
@jeherve jeherve requested a review from lancewillett March 9, 2018 11:54
@jeherve jeherve requested a review from a team as a code owner March 9, 2018 11:54
Copy link
Contributor

@lancewillett lancewillett left a comment

Choose a reason for hiding this comment

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

Great change, works as expected now.

Testing on iPhone 8 in mobile Safari, I see the dropdown menu instead of the overlay.

@lancewillett lancewillett added [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review This PR is ready for review. labels Mar 9, 2018
@dereksmart dereksmart merged commit 7009192 into master Mar 13, 2018
@ghost ghost removed the [Status] Ready to Merge Go ahead, you can push that green button! label Mar 13, 2018
@dereksmart dereksmart deleted the fix/translate-widget-default-style branch March 13, 2018 18:00
@oskosk oskosk added this to the 6.0 milestone Mar 15, 2018
oskosk added a commit that referenced this pull request Mar 23, 2018
dereksmart pushed a commit that referenced this pull request Mar 27, 2018
* Changelog 6.0: create base for changelog.

* Add #8938 to changelog

* Add #8962 to changelog

* Add #8974 to changelog

* Add #8975 to changelog

* Add #8978 to changelog

* Add #8867 to changelog

* Add #8937 to changelog

* Add #8961 to changelog

* Add #8855 to changelog

* Add #8944 to changelog

* Add #8973 to changelog

* Add #8977 to changelog

* Add #8979 to changelog

* Add #8980 to changelog

* Add #8982 to changelog

* Add #8983 to changelog

* Add #8984 to changelog

* Add #8986 to changelog

* Add #9005 to changelog

* Add #9010 to changelog

* Add #9012 to changelog

* Add #9021 to changelog

* Add #9022 to changelog

* Add #9056 to changelog

* Add #9061 to changelog

* Add #9079 to changelog

* Add #9080 to changelog

* Add #9088 to changelog

* Add #9096 to changelog

* Add #9097 to changelog

* Add #9100 to changelog

* Add #9107 to changelog

* Add #8969 to changelog

* Add #8993 to changelog

* Add #9003 to changelog

* Add #9031 to changelog

* Add #8945 to changelog

* Add #9052 to changelog

* Add #9058 to changelog

* Add #9066 to changelog

* Add #9076 to changelog

* Add #9053 to changelog

* Add #9108 to changelog

* Add #9135 to changelog

* Add #9148 to changelog

* Add #9125 to changelog

* Add #9137 to changelog

* Added testing instructions for 6.0.

* Added IS testing instructions, huge props to @tiagonoronha.

* Added #8498 to changelog.

* Added #8954 to changelog.

* Added #8985 to changelog.

* add #9027

* add #9112 to changelog

* add #9136 to changelog

* add #9102 to changelog

* add #9093 to changelog

* add #9062 to changelog

* add #9172 to changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Feature] Extra Sidebar Widgets [Pri] Normal [Type] Bug When a feature is broken and / or not performing as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Widgets: Google Translate widget popover isn't responsive

6 participants