- Issue created by @jessebaker
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
📌 Update Topbar and Navigator to better match designs Active is in!
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
🐛 The undo/redo doesn't always work Active also landed.
The undo and redo buttons do not feature in the designs and should be removed.
Postponed on
🐛
The undo/redo doesn't always work
Active
which updates tests around undo/redo
Postponed on
📌
Update Topbar and Navigator to better match designs
Active
which updates the top bar and also some of the undo/redo tests.
The component currently renders the buttons but crucially also binds the event listeners for the keyboard shortcuts so it needs to be refactored, not just removed.
The e2e tests frequently check the 'disabled' status of the undo and redo buttons. With them no longer in the Dom we need to ensure we are not missing test coverage.
Given the above, it may be preferable to relocated the buttons to somewhere else in the UI rather than remove them entirely. Will need to discuss with UX/Design.
The e2e tests should have been updated in 📌 Update Topbar and Navigator to better match designs Active to use keyboard shortcuts, but there may be some places missed.
Active
0.0
Page builder
📌 Update Topbar and Navigator to better match designs Active is in!
🐛 The undo/redo doesn't always work Active also landed.