Skip to content

Create headings for sidebar pages#1081

Merged
JuliaKirschenheuter merged 1 commit intomasterfrom
fix/35626-No_heading_nor_tab_index_for_settings_pages
Mar 8, 2023
Merged

Create headings for sidebar pages#1081
JuliaKirschenheuter merged 1 commit intomasterfrom
fix/35626-No_heading_nor_tab_index_for_settings_pages

Conversation

@JuliaKirschenheuter
Copy link
Contributor

@JuliaKirschenheuter JuliaKirschenheuter commented Feb 10, 2023

@nickvergessen
Copy link
Member

Where is the section defined. I assume a generic PR in server? How is ensured that it is not added when there is an h1 already.
Why are the words "Section" and "Subsection" added? That feels so wrong and unneeded.
If it would be helpful we could make activity filters have defining a "Activitiy by others" h2 title additional to the link name.
But the current wording really sound like a quirk/hack?🤔

@JuliaKirschenheuter
Copy link
Contributor Author

/backport to stable25

@JuliaKirschenheuter JuliaKirschenheuter marked this pull request as draft February 13, 2023 14:41
@JuliaKirschenheuter JuliaKirschenheuter force-pushed the fix/35626-No_heading_nor_tab_index_for_settings_pages branch 2 times, most recently from 65ca0af to 3b9e51a Compare March 2, 2023 10:55
@JuliaKirschenheuter JuliaKirschenheuter marked this pull request as ready for review March 2, 2023 10:57
@JuliaKirschenheuter
Copy link
Contributor Author

/backport to stable26

@nickvergessen nickvergessen removed their request for review March 7, 2023 06:19
Signed-off-by: julia.kirschenheuter <julia.kirschenheuter@nextcloud.com>
@JuliaKirschenheuter JuliaKirschenheuter force-pushed the fix/35626-No_heading_nor_tab_index_for_settings_pages branch from 6f96282 to eab9671 Compare March 8, 2023 07:20
@nickvergessen nickvergessen dismissed their stale review March 8, 2023 08:21

Dismissing

@JuliaKirschenheuter JuliaKirschenheuter merged commit 2048c62 into master Mar 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants