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

Fix-#20844:Styles-Fixed overlapping of text on closing button. #20845

Closed
wants to merge 1 commit into from

Conversation

Biki-das
Copy link
Collaborator

@Biki-das Biki-das commented Jan 19, 2022

This Pr Resolved the issue of overlapping text on the closing button.There were a lot of style changes which are confined to the
search Only.
Fixes-#20844

I have made good amount of css changes, so this has affected a few styles, lookig forward to maintainer reviews, if there any conflicting styles and changes to be made.

Below is the output after making these changes

design.-.Zulip.Dev.-.Zulip.-.Google.Chrome.2022-01-19.20-31-49.mp4

This Pr Resolved the issue of overlapping text on the closing button.There were a lot of style changes which are confined to the
search Only.
Fixes-zulip#20844
@anurastogiji
Copy link
Collaborator

@Biki-das @alya I have already worked on issue #19765 which is same as the issue @Biki-das opened ( #20844 )and I created a PR #20063 on which I worked for a loooong time following changes suggested by @alya.
It awaits review of @timabbott and is still open.

@Biki-das
Copy link
Collaborator Author

@timabbott need your review.

@alya
Copy link
Contributor

alya commented Jan 26, 2022

@Biki-das as discussed in PMs, please post a comment explaining how your solution differs from #20063 and #19790, and why, in #frontend on CZO. Please add a link to the thread here.

@zulipbot
Copy link
Member

Heads up @Biki-das, we just merged some commits that conflict with the changes your made in this pull request! You can review this repository's recent commits to see where the conflicts occur. Please rebase your feature branch against the upstream/main branch and resolve your pull request's merge conflicts accordingly.

@timabbott
Copy link
Sponsor Member

Closing as the original issue was completed in #20063.

@timabbott timabbott closed this Oct 2, 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.

5 participants