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

feat(theming): Provide body-container- variables through theming #46259

Merged
merged 2 commits into from
Jul 7, 2024

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Jul 2, 2024

Summary

  • Provide the body container height, radius and margin as theming variables.
  • Remove unused old variables (header-menu-profile-item-height and list-max-width)
  • Fix comment about breakpoint JS location

I will create a documentation PR as soon as the ground work is merged ( nextcloud/documentation#11952 )

Checklist

@susnux susnux added enhancement 3. to review Waiting for reviews labels Jul 2, 2024
@susnux susnux requested review from marcoambrosini, a team, Fenn-CS, Pytal, sorbaugh and artonge and removed request for a team and Pytal July 2, 2024 20:33
@susnux susnux added this to the Nextcloud 30 milestone Jul 2, 2024
@susnux susnux force-pushed the feat/provide-body-container branch from ac54818 to 16d0c82 Compare July 2, 2024 20:49
@susnux susnux requested a review from skjnldsv July 3, 2024 12:05
@susnux susnux force-pushed the feat/provide-body-container branch from 16d0c82 to 6925876 Compare July 3, 2024 12:06
* Provide the body container height, radius and margin as theming variables.
* Remove unused old variables (`header-menu-profile-item-height` and `list-max-width`)
* Fix comment about breakpoint JS location

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the feat/provide-body-container branch from 6925876 to 6ef88de Compare July 4, 2024 16:29
@susnux susnux merged commit 03b1918 into master Jul 7, 2024
165 checks passed
@susnux susnux deleted the feat/provide-body-container branch July 7, 2024 12:24
susnux added a commit to nextcloud/documentation that referenced this pull request Jul 7, 2024
* For nextcloud/server#46259

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@blizzz blizzz mentioned this pull request Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants