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.

Sign up for GitHub

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

Jump to bottom

BUG: Layer visibility does not work properly #3508

Closed
tranthanhhoa opened this issue Jun 1, 2021 · 1 comment
Closed

BUG: Layer visibility does not work properly #3508

tranthanhhoa opened this issue Jun 1, 2021 · 1 comment
Labels
bug

Comments

@tranthanhhoa
Copy link

Version: 0.17.4

How to reproduce

  1. Switch to tablet or mobile mode
  2. Hide the selected component by toggling the visibility icon of Layer
  3. Switch to desktop mode.

Result

  • The visibility icon is still disabled while the hidden components is showing.

Expected

  • After switch to desktop the visibility icon is enabled.
@artf artf added the bug label Jun 21, 2021
@artf
Copy link
Member

artf commented Jun 21, 2021

Thanks for the report @tranthanhhoa I'll fix it in the next release, but I think I'll keep the visibility change more like a global condition, so the hidden component won't be visible on all devices.

@artf artf closed this as completed in e7e91e8 Jun 21, 2021
felixx-kreebox pushed a commit to kreeboxvn/grapesjs that referenced this issue Feb 28, 2022
@artf @felixx-kreebox
Fix layers visibility change. Closes GrapesJS#3508
68c594e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tranthanhhoa @artf