Skip to content

[VK] Updated global header component with some additional required styles - #584

Open
3697varun wants to merge 8 commits into
mainfrom
Updated_global_header_package_with_some_required_css
Open

[VK] Updated global header component with some additional required styles#584
3697varun wants to merge 8 commits into
mainfrom
Updated_global_header_package_with_some_required_css

Conversation

@3697varun

Copy link
Copy Markdown

Updated some essential style for common global header package with comments

@3697varun
3697varun requested a review from jon-stevens October 13, 2021 10:05
position: relative;
display: flex;
justify-content: space-between;

@jon-stevens jon-stevens Oct 13, 2021

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The flex rule kicks in with the media query just below.
the following should be added to the README

@import '@springernature/brand-context/default/scss/10-settings/breakpoints';
@import '@springernature/brand-context/default/scss/30-mixins/media-query';```

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Changes are done in README

@import '@springernature/springernature-header/scss/10-settings/layout';
@import '@springernature/springernature-user-details/scss/10-settings/typography'; // if including user-details data
@import '@springernature/springernature-user-details/scss/50-components/core'; // if including user-details data
@import '@springernature/brand-context/springernature/scss/10-settings/typography';

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This import should probably occur above scss/10-settings/typography

Comment thread toolkits/springernature/packages/springernature-header/README.md
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.

3 participants