Overview

Lastest designs move the zoom controls and show just one viewport at a time which can be switched between desktop and mobile.

Proposed resolution

This might require exploration on how to ensure the changes to the canvas is as user friendly as possible. The designs introduce two new buttons to center and scale-to-fit.

One idea I have to make sure that when switching between viewports the currently selected component is scrolled into view to make comparing as easy as possible.

User interface changes

CommentFileSizeAuthor
zoom_and_breakpoint_toggle.png90.6 KBjessebaker
Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

jessebaker created an issue. See original summary.

jessebaker’s picture

Adding https://www.drupal.org/project/experience_builder/issues/3481658 as a related issue - would be nice to get the mouse cursor centering in along with this change!

jessebaker’s picture

jessebaker’s picture

Status: Active » Needs review

bnjmnm made their first commit to this issue’s fork.

  • bnjmnm committed d300832e on 0.x authored by jessebaker
    Issue #3522209 by jessebaker: Move zoom controls, show only one viewport
    
bnjmnm’s picture

Assigned: jessebaker » Unassigned
Status: Needs review » Fixed

This looks good and there are enough related followups happening to clean up any loose ends in MRs with a more manageable number of files changed.

wim leers’s picture

wim leers’s picture

While our heads are in this space, can we efficiently tackle (title-wise related) #3473222: [test-only] Need to update the position of contextual edit component menu while zooming out/in in the canvas too?

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.