Skip to content

Conversation

@kunwp1
Copy link
Contributor

@kunwp1 kunwp1 commented Oct 24, 2024

This PR improves the user interface by updating the message displayed when a user selects files to create a dataset. Previously, the message read "# files uploaded successfully!" after the files were selected from the local machine. However, based on feedback from @chenlica, the term "uploaded" was found to be misleading, as the files are not yet uploaded to the server at this stage.

To clarify the process, this PR changes the wording from "uploaded" to "selected," making it clear that the files have only been chosen locally and have not yet been transferred to the server.

Additionally, I added a conditional expression to determine whether to use "file" or "files" based on the count.

[Before the change]
Screenshot 2024-10-24 at 11 17 02 AM

[After the change]
Screenshot 2024-10-24 at 11 20 57 AM

@kunwp1 kunwp1 added the gui label Oct 24, 2024
@kunwp1 kunwp1 self-assigned this Oct 24, 2024
Copy link
Contributor

@bobbai00 bobbai00 left a comment

Choose a reason for hiding this comment

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

lgtm!

@kunwp1 kunwp1 merged commit 5a4aec3 into master Oct 24, 2024
@kunwp1 kunwp1 deleted the chris-fix-dataset-upload-message branch October 24, 2024 18:38
PurelyBlank pushed a commit that referenced this pull request Dec 4, 2024
This PR improves the user interface by updating the message displayed
when a user selects files to create a dataset. Previously, the message
read "# files uploaded successfully!" after the files were selected from
the local machine. However, based on feedback from @chenlica, the term
"uploaded" was found to be misleading, as the files are not yet uploaded
to the server at this stage.

To clarify the process, this PR changes the wording from "uploaded" to
"selected," making it clear that the files have only been chosen locally
and have not yet been transferred to the server.

Additionally, I added a conditional expression to determine whether to
use "file" or "files" based on the count.

[Before the change]
<img width="711" alt="Screenshot 2024-10-24 at 11 17 02 AM"
src="https://github.com/user-attachments/assets/343b0218-d669-4cfd-9175-cf58f362416b">

[After the change]
<img width="550" alt="Screenshot 2024-10-24 at 11 20 57 AM"
src="https://github.com/user-attachments/assets/55f2f537-6bf7-4136-9957-47c964d08f0e">

---------

Co-authored-by: Kunwoo Park <kunwoopark@dhcp-172-31-218-249.mobile.uci.edu>
@Yicong-Huang Yicong-Huang added frontend Changes related to the frontend GUI and removed gui labels Jul 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend Changes related to the frontend GUI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants