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

Remove multiple buttons with same functionality in dev workbench #1068

Conversation

chandrikarj
Copy link
Contributor

Remove duplicate "Choose file" buttons and consolidate functionality in Workbench UI

Summary

This PR removes redundant "Choose file" buttons across the Workbench UI, ensuring a cleaner and more intuitive design. A single "Choose file" button is now shared dynamically between the dataset selection and custom data upload sections. The UI has been refactored to avoid duplication while maintaining all required functionality.

Before:

Multiple "Choose file" buttons appeared across different sections and modals.
Confusion for users due to redundant UI elements.
After:

Only one "Choose file" button dynamically serves all use cases.
Simplified UI improves user experience and reduces visual clutter.
Screenshots:

Before:
Screenshot 2025-01-13 at 2 58 44 PM
Screenshot 2025-01-13 at 8 11 27 PM
Screenshot 2025-01-13 at 8 22 39 PM

After:
Screenshot 2025-01-13 at 8 13 51 PM
Screenshot 2025-01-13 at 8 13 58 PM
Screenshot 2025-01-13 at 8 14 03 PM

Motivation

The previous design had multiple "Choose file" buttons that served the same functionality in different contexts. This redundancy led to a cluttered UI and potential confusion for users. This PR simplifies the interface by consolidating the functionality into a single button that dynamically adapts to its context.

Testing

The changes have been tested by:

Ensuring the "Choose file" button dynamically updates based on the selected context (dataset selection or custom data upload).
Verifying that the selected file is correctly displayed in the corresponding section or modal.
Testing across multiple browsers (Chrome, Firefox, Safari) to ensure consistent functionality.

@birm birm merged commit e640ee3 into camicroscope:develop Jan 23, 2025
1 of 2 checks passed
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.

2 participants