-
Notifications
You must be signed in to change notification settings - Fork 146
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
Specify content density handling with Fiori Launchpad in more details #172
base: main
Are you sure you want to change the base?
Conversation
Describe default values for compact/cozy content density more specifically
|
Hi @ulasenka , thanks for your contribution! We'll look into it and will get back to you. |
docs/03_Get-Started/step-36-content-density-typescript-667aa4a.md
Outdated
Show resolved
Hide resolved
docs/03_Get-Started/step-36-content-density-typescript-667aa4a.md
Outdated
Show resolved
Hide resolved
@@ -10,7 +10,7 @@ In this step of our Walkthrough tutorial, we adjust the content density based on | |||
|
|||
|
|||
|
|||
**The content density is compact on desktop devices and cozy on touch-enabled devices** | |||
**The default content density is cozy on touch devices (smartphones, tablets and computers with a touch screen) and compact on devices without a touch screen. Note that Fiori Launchpad sets cozy content density on phones and tablets as mandatory and may override it on other computers according to the application manifest and the user preference.** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ulasenka Is there a reason why this only is added for the JavaScript text but not to the TypeScript step?
@margopolo @petermuessig The Walkthrough tutorials demonstrate how to build a standalone app, so there is no outer shell like the SAP Fiori launchpad that could have set the content density class already. What do you think - should we leave the Walkthroughs as they are then? Or should we mention in the text that care needs to be taken if an app is embedded in the FLP? |
No description provided.