element-hq / element-web

A glossy Matrix collaboration client for the web.
https://element.io
GNU Affero General Public License v3.0
11.17k stars 1.99k forks source link

Fix 'remove alias' button in room settings #28269

Closed Dev-Gurjar closed 3 days ago

Dev-Gurjar commented 1 week ago

Checklist

Closes https://github.com/element-hq/element-web/issues/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 commented 1 week ago

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

Dev-Gurjar commented 1 week ago

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 commented 6 days ago

@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 commented 5 days ago

is it working now?