Skip to content

Conversation

MichalBryxi
Copy link
Contributor

  • The package insists on having @ember/string v3
  • It does not do too much

- The package insists on having @ember/string v3
- It does not do too much
{{if this.media.isMobile
(if this.isShowingMenu "docs-opacity-100 docs-bg-white" "docs-opacity-0 docs-pointer-events-none")
}}
{{if this.isShowingMenu "docs-opacity-100 docs-bg-white md:docs-opacity-0" "docs-opacity-0 md:docs-opacity-100 docs-pointer-events-none md:docs-pointer-events-auto"}}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think the truthy part of this can be ommited as it won't be shown anyways as the button toggling isShowingMenu won't be there. But for a good measure I added the opacity styles anyway.

@MichalBryxi
Copy link
Contributor Author

Ready for review @SergeAstapov

@SergeAstapov SergeAstapov merged commit 6c7633e into ember-learn:main May 3, 2025
15 checks passed
@MichalBryxi MichalBryxi deleted the mb/1675-remove-ember-responsive branch May 3, 2025 07:03
@MichalBryxi
Copy link
Contributor Author

Fixed: #1675

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants