Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request aims to fix a spacing issue within the file-uploader component by refactoring the HTML structure. The changes are generally well-implemented and focused. I have identified a couple of minor areas for improvement: one regarding redundant CSS classes which can affect maintainability, and another concerning a small violation of the project's UX writing guidelines. My review comments provide specific suggestions to address these points.
projects/element-ng/file-uploader/si-file-uploader.component.html
Outdated
Show resolved
Hide resolved
projects/element-ng/file-uploader/si-file-uploader.component.html
Outdated
Show resolved
Hide resolved
spike-rabbit
left a comment
spike-rabbit
left a comment
There was a problem hiding this comment.
Looks already better, just the alignment of the validation icon seem to be off.
I left some comments which should fix it.
Can you please also check if the AI comments are valid.
Please also choose a more descriptive commit message. Something about align with UX specs or so.
projects/element-ng/file-uploader/si-file-uploader.component.html
Outdated
Show resolved
Hide resolved
projects/element-ng/file-uploader/si-file-uploader.component.html
Outdated
Show resolved
Hide resolved
projects/element-ng/file-uploader/si-file-uploader.component.html
Outdated
Show resolved
Hide resolved
|
@hbxes can you please have a look here. |
robertwilde
force-pushed
the
fix/file-uploader/spacing
branch
from
621f9d0 to
c314a42
Compare
robertwilde
force-pushed
the
fix/file-uploader/spacing
branch
from
c314a42 to
fa15d8f
Compare
3 participants
Related to #1412
Fix spacing between warning/error label and upper two labels as specified in the Figma source.
Documentation.
Examples.
Dashboards Demo.
Playwright report.
Coverage Reports: