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 'remove alias' button in room settings #28269

Merged
merged 6 commits into from
Oct 28, 2024

Conversation

Dev-Gurjar
Copy link
Contributor

@Dev-Gurjar Dev-Gurjar commented Oct 22, 2024

Checklist

  • Tests written for new code (and old code if feasible).
  • New or updated public/exported symbols have accurate TSDoc documentation.
  • Linter and other CI checks pass.
  • I have licensed the changes to Element by completing the Contributor License Agreement (CLA).

Closes #27985

Before and After Comparison:

Before: After:

Explanation:

The size of the button had to be increased to prevent the "close room settings" button from breaking. The images above show the difference before and after the changes, with the button's size adjustment ensuring proper functionality and alignment.

@Dev-Gurjar Dev-Gurjar requested a review from a team as a code owner October 22, 2024 17:48
@github-actions github-actions bot added the Z-Community-PR Issue is solved by a community member's PR label Oct 22, 2024
@Dev-Gurjar
Copy link
Contributor Author

this is my first open source contribution please tell me what am i doing wrong?

Copy link
Member

@florianduros florianduros left a comment

Choose a reason for hiding this comment

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

Hi! Thanks for your contribution!

As you can see in the result of the e2e test (e2e-28269--matrix-react-sdk.netlify.app), the css changes introduce a regression on the dialog close button.

@Dev-Gurjar
Copy link
Contributor Author

Dev-Gurjar commented Oct 24, 2024

I have made the change, and this should fix that.
changes - 0025ee7

Is it possible to check by myself if all the errors are solved?

@florianduros
Copy link
Member

florianduros commented Oct 25, 2024

@Dev-Gurjar

Click on View deployment of the e2e test line:
image

or

The check mark is green in the CI check
image

@Dev-Gurjar
Copy link
Contributor Author

is it working now?

@florianduros florianduros changed the title fixed 'remove alias' button Fix 'remove alias' button in room settings Oct 28, 2024
Copy link
Member

@florianduros florianduros left a comment

Choose a reason for hiding this comment

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

Tested and working! Thanks for the contribution.

@florianduros florianduros added this pull request to the merge queue Oct 28, 2024
Merged via the queue into element-hq:develop with commit a2c7870 Oct 28, 2024
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Defect Z-Community-PR Issue is solved by a community member's PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Visually broken 'remove alias' button
2 participants