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

Widget editor: In customizer, wide, tall widgets display poorly #32973

Closed
dreamwhisper opened this issue Jun 24, 2021 · 4 comments · Fixed by #32976
Closed

Widget editor: In customizer, wide, tall widgets display poorly #32973

dreamwhisper opened this issue Jun 24, 2021 · 4 comments · Fixed by #32976
Assignees
Labels
[Feature] Widgets Customizer Ability to add and edit blocks in Customize → Widgets. [Type] Bug An existing feature does not function as intended

Comments

@dreamwhisper
Copy link

dreamwhisper commented Jun 24, 2021

Description

Step-by-step reproduction instructions

Please list the steps needed to reproduce the bug. For example:

  1. Get Genesis & Genesis Sample
  2. Go to Customizer > Widgets
  3. Add a Genesis - Featured Posts widget to a widget area.
  4. Note the position and whether all settings are accessible. Settings may be cut off on the top or bottom depending on the position of the widget.
  5. Note the width of the widget and field display. It is narrower than the non-block-based widget areas.

Expected behaviour

Legacy widget settings will always be accessible & form will display similarly to previous display.

If the height wasn't enough, the widget was scrollable in the non-block-based widget areas in the customizer. (It's not currently scrollable due to #32969.)

Actual behaviour

The Genesis - Featured Post widget is:

  • Narrower, so settings don't fit nicely.
  • Positioned too high or too low to see or reach all settings.
  • Not scrollable so settings are reachable.

Screenshots or screen recording (optional)

Original:

Screen Shot 2021-06-24 at 3 48 55 PM

Block-based at the top:

Screen Shot 2021-06-24 at 3 08 46 PM

Block-based pushed down:

Screen Shot 2021-06-24 at 3 23 33 PM

Even if we were to fix #32969 in Genesis Framework, the position may not be correct:

Screen Shot 2021-06-24 at 5 37 26 PM

Related

WordPress information

  • WordPress version: 5.8-beta3-51224
  • Gutenberg version: Not installed"
  • Are all plugins except Gutenberg deactivated? Yes
  • Are you using a default theme (e.g. Twenty Twenty-One)? No - Get Genesis

Device information

  • Device: Mac
  • Operating system: MacOS 10.15.x
  • Browser: Chrome 91
@dreamwhisper dreamwhisper changed the title Widget editor: In customizer, wide widgets display poorly Widget editor: In customizer, wide, tall widgets display poorly Jun 24, 2021
@johnstonphilip johnstonphilip added [Feature] Widgets Customizer Ability to add and edit blocks in Customize → Widgets. [Type] Bug An existing feature does not function as intended labels Jun 25, 2021
@talldan
Copy link
Contributor

talldan commented Jun 25, 2021

Thanks for reporting this! #31736 was the previous improvement to wide widgets, looks like a few additional adjustments will be needed.

@tellthemachines tellthemachines self-assigned this Jun 25, 2021
@tellthemachines
Copy link
Contributor

There are two separate issues here:

  1. The popover is partially cut off and not scrollable - this is due to the popover not containing the floated contents, and can be fixed easily.
  2. The popover is too narrow - this is due to the widget itself not defining its width. This should be fixed in the theme/plugin supplying the widget, as wide widgets can come in many shapes and sizes so it would not be appropriate to set a width for them in the Customizer. At best we can set a min-width, but this is unlikely to work well for all cases.

@johnstonphilip
Copy link
Contributor

@tellthemachines If an old plugin has to update its code (like to set widgets widths) in order to fix a new problem that never existed before, than this is not really backwards compatible, no?

@tellthemachines
Copy link
Contributor

@johnstonphilip backward compatibility is ensured to the extent that these widgets are now usable, and we have also made it possible for themes to opt out of the new widgets editor.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Widgets Customizer Ability to add and edit blocks in Customize → Widgets. [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants