Merged
Conversation
AlexAndBear
reviewed
Feb 28, 2025
Comment on lines
+40
to
+41
| --ifm-hr-margin-vertical: 4rem; | ||
| --ifm-leading: calc(var(--ifm-leading-desktop)* 1.25rem); |
Member
There was a problem hiding this comment.
No frickelei, nice, I like I like 👍
Member
|
@tbsbdr Are we sure about the emojis in the left side bar? In order to make it equal trough the whole ui, that would require an emoji for all categories 
|
Member
Author
|
no, not sure at all. trying to get some feedback: https://matrix.to/#/!MXIMfhTMMRFPuXSKJZ:matrix.org/$9iWXcfsHiL0FJsJlYKxkiEk30jHD3aQXf6KI6ILwCQs?via=chat.opencloud.eu&via=matrix.org&via=tchncs.de |
Member
Author
|
we decided to remove the emojis from the navbar and find some more uniform markers like eg. the bult-in icons in the future. for now lets merge without the emojis. |
AlexAndBear
approved these changes
Feb 28, 2025
Expected outcome: - give first time users a warm welcome and the feeling of "yes, I can do that" (do not overwhelm eg. with complex 1st level navitems) - provide a low-threshold entry to the docs for first time users Changes summary - Simplified sidebar navigation - Visual focus on main content that reflects the admin install journey: 1. Install -> 2. Configure -> 3. Maintain - wrapper: *Welcome* and *Resources* for anything else - Added planned pages as drafts so that we can see the bigger picture - Organized folders and pages with fixed-ordering for easier internal orientation (has no impact for endusers) - Split up "Requirements" into dedicated navigation item for better findability as this is frequently requested and might not be assumed within the datasheet. - increased leading and <hr> margin to increase whitespace so that I could remove <br> that where used for styling purposes - Added Emojis to be more welcoming; we are no IBM, lets be friendly.
- removed icons from navbar - added correct curl url - added descriptions to getting started pages so that the cards on the autogenerated overview pages are more descriptive - added `coming soon`
tbsbdr
force-pushed
the
Preset-for-Navigation-Structure
branch
from
e8b45e7 to
8dd3ef3
Compare
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Expected outcome
Changes summary
<hr>margin to increase whitespace so that I could remove<br>that where used for styling purposesVisuals
Screenie
Video
screenie000233-converted.mp4