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

No spacing between tooltips of organisations #1098

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Ayushsunny
Copy link
Collaborator

@Ayushsunny Ayushsunny commented Apr 3, 2021


What's this PR do?
Fixed spacing between the tooltip of organisation and the sidebar in #1087

Any background context you want to provide?
Solved the issue without any conflicts.

Screenshots?

Before commit
before

After commit

Screenshot from 2021-04-04 10-29-17

You have tested this PR on:

  • Windows
  • Linux(Ubuntu 20.04 - Linux x86_64)
  • macOS

@Ayushsunny
Copy link
Collaborator Author

Hello @timabbott , @andersk could you please review and merge ?

@Ayushsunny Ayushsunny changed the title Solved Conflict No spacing between tooltips of organisations Apr 3, 2021
@andersk
Copy link
Member

andersk commented Apr 4, 2021

This appears to be the same change as #1091. Please don’t close and open multiple PRs for the same change—update the existing PR instead.

Your commit message “Solved Conflict” gives no information at all about what you are changing and why. See our documentation for guidance on commit messages.

@Ayushsunny
Copy link
Collaborator Author

Your commit message “Solved Conflict” gives no information at all about what you are changing and why. See our documentation for guidance on commit messages.

Sure, Sir, I will edit it.

@zulipbot
Copy link
Member

zulipbot commented Apr 9, 2022

Heads up @Ayushsunny, 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.

@Kamlesh72
Copy link

is this issue still pending?

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