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

feat: Enhance fileUpload modal. #899

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

thepiyush-303
Copy link
Contributor

@thepiyush-303 thepiyush-303 commented Jan 14, 2025

Brief Title

In this PR i work on enhancing the FileUpload modal and make it look similar as RC as much as possible
Edited: In a file upload modal when you click on label it does not focus on the input field i have added this in my PR.

  • it does not show error about file name is required.
  • when the file name is empty it was allowing you to upload but it does not send that file.
  • when you get large photos(large height) then the modal size does not increase.

Acceptance Criteria fulfillment

  • adding error msg if name field is empty.
  • access input fields through labels.
  • set modal-height to auto.

Fixes #891

Video/Screenshots

Before:

Screencast.from.2025-01-15.00-33-09.mp4

After:

Screencast.from.2025-01-15.00-31-17.mp4

PR Test Details

Note: The PR will be ready for live testing at https://rocketchat.github.io/EmbeddedChat/pulls/pr-899 after approval. Contributors are requested to replace <pr_number> with the actual PR number.

@Spiral-Memory
Copy link
Collaborator

Please use mp4 formats and clearly explain the difference in the PR description

@thepiyush-303
Copy link
Contributor Author

@Spiral-Memory

Done.

@Spiral-Memory
Copy link
Collaborator

when you get large photos(large height) then the modal size does not increase

Let's keep the modal size fixed

@thepiyush-303
Copy link
Contributor Author

@Spiral-Memory
Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix: Enhance fileUpload modal and fixing behaviour of modals.
2 participants