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.

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

style(cypht): scope menu toggling to not interfere with Tiki elements #1381

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

josaphatim
Copy link
Member

Click on menu button is having impact on Tiki menu as same html tags involved are shared in both Cypht and Tiki.

@josaphatim josaphatim changed the title Scope menu toggling to not interfere with Tiki elements cypht: Scope menu toggling to not interfere with Tiki elements Nov 23, 2024
@josaphatim josaphatim changed the title cypht: Scope menu toggling to not interfere with Tiki elements cypht: scope menu toggling to not interfere with Tiki elements Nov 23, 2024
@josaphatim josaphatim changed the title cypht: scope menu toggling to not interfere with Tiki elements style: scope menu toggling to not interfere with Tiki elements Nov 23, 2024
@josaphatim josaphatim changed the title style: scope menu toggling to not interfere with Tiki elements style(cypht): scope menu toggling to not interfere with Tiki elements Nov 23, 2024
@mercihabam
Copy link
Member

mercihabam commented Nov 23, 2024

@josaphatim, to achieve your intended behavior, you need to add the cypht-layout class here: https://github.com/cypht-org/cypht/blob/master/modules/core/site.css#L1464

Some of the current changes might still be interesting to keep!

@josaphatim josaphatim force-pushed the prefix-elements-to-not-interfere-in-intergration branch from 661f812 to d970b7d Compare November 23, 2024 16:52
@josaphatim
Copy link
Member Author

@jacob-js I like the proposed solution. Other changes were committed by error. It's a work in progress

@josaphatim josaphatim marked this pull request as draft November 23, 2024 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants