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

docs: Oryx's Responsive Design system #2214

Merged
merged 23 commits into from
Oct 24, 2023

Conversation

tobi-or-not-tobi
Copy link
Contributor

@tobi-or-not-tobi tobi-or-not-tobi commented Oct 3, 2023

Learn how to leverage Oryx's Responsive Design system to create stunning and user-friendly websites and applications that seamlessly adapt to different screen sizes and devices.

closes HRZ-2164

@Sinucid Sinucid self-requested a review October 5, 2023 11:40
Sinucid
Sinucid previously approved these changes Oct 5, 2023

While the `xs` and `xl` sizes are available in the TypeScript enumeration, they are not used in most Oryx components and layouts. However, you can use the definitions to create an optimized user experience for these screen sizes. Moreover, you can introduce additional screen sizes if needed.

The actual screen definition is provided by a breakpoint configuration and can be referenced by their code. If you need to use the screen definitions in your custom code, you can use `ScreenService` provided by the [layout package](https://www.npmjs.com/package/@spryker-oryx/layout).
Copy link
Collaborator

Choose a reason for hiding this comment

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

Who are you referencing to by their?

@andriitserkovnyi andriitserkovnyi added the new PR contains a new document label Oct 17, 2023
### Inline stylesheets

In the following example, style rules are added for small screens (`sm`) only.
The disadvantage of adding styles to the component definition is the instant effect when the application is bootstrapped. In most cases, we recommend dynamically importing the styles so that they're only loaded when the component is used.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could you elaborate on the instant effect? I feel like there is something wrong here, but need more info to figure it out

@andriitserkovnyi andriitserkovnyi merged commit c23ad68 into master Oct 24, 2023
13 checks passed
@andriitserkovnyi andriitserkovnyi deleted the docs/oryx-responsive-design branch October 24, 2023 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new PR contains a new document
Development

Successfully merging this pull request may close these issues.

3 participants