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

stream settings: Show stream header in narrow view. #22965

Merged
merged 1 commit into from Oct 13, 2022

Conversation

juliaBichler01
Copy link
Collaborator

This changes the streams overlay so that the header with the stream name or with 'Create stream' is also visible in a narrow window.

Resolves #22963

tmp

tmp2

This changes the streams overlay so that the header with the stream name or
with 'Create stream' is also visible in a narrow window.

Resolves zulip#22963
@alya
Copy link
Contributor

alya commented Sep 17, 2022

Nice! @ganpa3 would you be up for reviewing this one?

@ganpa3
Copy link
Collaborator

ganpa3 commented Sep 17, 2022

LGTM.

@alya
Copy link
Contributor

alya commented Sep 19, 2022

Great! This small change has been approved by @ganpa3 , and looked good to me in manual testing. @timabbott please take a look. :)

@juliaBichler01 juliaBichler01 changed the title stream settings: Show stream header in narrow view. stream-settings: Show stream header in narrow view. Oct 3, 2022
@juliaBichler01 juliaBichler01 changed the title stream-settings: Show stream header in narrow view. stream settings: Show stream header in narrow view. Oct 3, 2022
@alya alya added the integration review Added by maintainers when a PR may be ready for integration. label Oct 3, 2022
@alya
Copy link
Contributor

alya commented Oct 3, 2022

Oops, looks like I forgot to add the "integration review" label earlier!

#groups_overlay {
.display-type {
display: none;
}
}
Copy link
Sponsor Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@m-e-l-u-h-a-n I'm not sure whether the group_overlay block here should be removed as well; can you test that and submit a quick PR if appropriate?

@timabbott timabbott merged commit 97fd443 into zulip:main Oct 13, 2022
@timabbott
Copy link
Sponsor Member

Merged, thanks @juliaBichler01!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
integration review Added by maintainers when a PR may be ready for integration. size: XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Stream name header not visible at narrow widths in the Manage stream overlay.
5 participants