Skip to content

FEATURE: enable topic creation button for read only categories #34211

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

Merged
merged 6 commits into from
Aug 14, 2025

Conversation

dbattersby
Copy link
Contributor

@dbattersby dbattersby commented Aug 11, 2025

Most of this feature was already approved in #33495 but it was reverted temporarily due to users accidentally posting in the wrong category (due to the default composer category site setting).

This PR removes the default category from composer when the user doesn't have permission to post in the category (preventing accidental posts in the wrong category), and also inserts the read-only category into the category chooser with a read only badge, this way providing some information to the user when they find it in the dropdown.

The read only category in the drop down can't be selected via clicking or keyboard selection.

Screenshot 2025-08-12 at 6 33 34 PM

@github-actions github-actions bot added the i18n PRs which update English locale files or i18n related code label Aug 11, 2025
@dbattersby dbattersby force-pushed the always-enable-topic-creation-btn branch from a34ef7c to 7c69ed8 Compare August 12, 2025 13:08
@dbattersby dbattersby marked this pull request as ready for review August 12, 2025 14:34
@dbattersby dbattersby merged commit 44bf488 into main Aug 14, 2025
17 checks passed
@dbattersby dbattersby deleted the always-enable-topic-creation-btn branch August 14, 2025 07:19
dbattersby added a commit that referenced this pull request Aug 14, 2025
dbattersby added a commit that referenced this pull request Aug 14, 2025
#34334)

Revert changes in #34211 temporarily until we have updated the separate
sidebar new topic btn theme component.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i18n PRs which update English locale files or i18n related code
Development

Successfully merging this pull request may close these issues.

2 participants