-
Notifications
You must be signed in to change notification settings - Fork 0
Release 0.3.4 #571
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
Open
golovencoserghei
wants to merge
23
commits into
main
Choose a base branch
from
release-0.3.4
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Release 0.3.4 #571
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Removed redundant and unused variables from GalleryLayoutElement to improve code clarity and maintainability. This cleanup eliminates unnecessary json_encode and BrizyComponent instantiations.
…GalleryLayoutElement refactor: remove unused variables in GalleryLayoutElement
Introduced a method to calculate additional top padding for detail pages. Updated constructor to handle exceptions with proper logging and improved item transformation logic for MediaLayoutElement.
…-detail-pages feat: add support for additional top padding in detail pages
Adjusted `hoverViewColorOpacity` and `viewColorOpacity` values to improve consistency. Added new `activeViewColor*` properties to enhance layout customization options. These changes aim to provide better control over event layout styling.
Introduce `getThemeMenuItemActiveSelector` method across themes to define selectors for active menu items. This ensures consistency in handling active state styles and improves navigation functionality.
…for-active-and-inactive-menu-items feat: add active menu item selector for theme navigation
fix(menu): active color is wrong for menu item
Introduce functionality to extract and handle deep slugs and full URLs. This ensures dynamic browser page opening based on menu and migration contexts, improving flexibility during layout transformations.
…active-sub-menu-items feat: add support for dynamic page URL handling in migration
fix(text): missing header text for solstice theme
…t-widgetelement feat(event-layout): update color opacity and add active view
Replaced slug with id as the parameter for generating full URLs during page mapping. This ensures correct URL resolution by aligning with the intended method signature and logic.
…ge-url fix(page-mapping): correct parameter usage in URL generation
fix(menu): hover background-color is not applied
fix(font-size): added precision to two digits for font-size
Added PathSlugExtractor and ElementContextInterface dependencies to the Head element to improve functionality. Removed unnecessary blank lines in AbstractElement methods for cleaner code structure.
…ice-Head-element refactor: move browser page logic to Solstice Head element
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.