-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
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
archive stream: Update modal text. #22662
Conversation
Hello @zulip/server-streams members, this pull request was labeled with the "area: stream settings" label, so you may want to check it out! |
Nice! I think we need more space before and after the stream symbols. @amanagr I remember you working on that sort of thing -- do you have any helpful pointers? |
Looking at those screenshots, let's also remove the word "stream" from the heading; I don't think we need it after the stream symbol has been added. |
df5bc8a
to
ed8d48c
Compare
Done. |
We need some space between the privacy icon and the stream name. Also |
Done. Updated the PR description. |
ed8d48c
to
d14a63a
Compare
Merged, thanks @ganpa3! |
Fixes #22658.
Screenshots and screen captures:
Private stream
Web public stream
Public stream