Skip to content

fix: Remove ember-responsive #1676

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

Merged

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