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

chore: UI/UX improvements #3319

Merged
merged 9 commits into from
Jan 5, 2024
Merged

chore: UI/UX improvements #3319

merged 9 commits into from
Jan 5, 2024

Conversation

prateekshourya29
Copy link
Member

This PR fixes several UI/UX issue in the platform.

Problems Fixed

1. Progress chart error message is incorrect for cycle/ module that are not active yet.

The upcoming Cycles/Modules still display Invalid Date despite having start and end dates specified.
image

Solution

Updated it to This cycle/module isn't active yet.
image

2. After updating the workspace from workspace settings. The loader isn’t stopping.

The issue occured because we updated the global state with incomplete form data, lacking crucial fields like the workspace slug. Consequently, the currentWorkspace remained null as we couldn't filter its details without the necessary slug in the global state.

Solution

Updated the updateWorkspace action to use response data which contains the complete object.

workspace_update.1.mp4

3. In user profile settings, the email field should have cursor-not-allowed style.

The email field cannot be edited, but its appearance lacks any distinguishing style, making it seem editable.

Solution

Added proper Disabled style to it.
image

4. User profile icon dropdown doesn't closes automatically.

The dropdown for the Profile icon in the sidebar header doesn't close when any menu item is clicked.

Solution
user_dropdown.1.mp4

5. Inconsistent padding in Cycle empty state.

Solution
cycle_padding.1.mp4

6. Project labels empty state showing two times. Also, empty state should be removed if I click on add label.

7. Inconsistent padding in Labels, Integration, Estimates settings empty state.

Solution
labels.1.mp4

8. Workspace Integration setting breadcrumb shows Export Settings.

image

Solution

image

image={emptyIntegration}
primaryButton={{
text: "Configure now",
onClick: () => router.push(`/${workspaceSlug}/settings/integrations`),

Check warning

Code scanning / CodeQL

Client-side URL redirect Medium

Untrusted URL redirection depends on a
user-provided value
.
@sriramveeraghanta sriramveeraghanta merged commit 9dd8c8b into develop Jan 5, 2024
6 of 8 checks passed
@sriramveeraghanta sriramveeraghanta deleted the chore/ui-improvements branch January 5, 2024 08:43
sriramveeraghanta pushed a commit that referenced this pull request Jan 22, 2024
* chore: add proper message for cycle/ module having start & end date but isn't active yet.

* fix: infinite loader after updating workspace settings.

* fix: user profile icon dropdown doesn't closes automatically.

* style: fix inconsistent padding in cycle empty state.

* chore: remove multiple `empty state` in labels settings and improve add label logic.

* style: fix inconsistent padding in project label, integration and estimates empty state.

* style: fix integrations settings breadcrumb title.

* style: add proper `disabled` styles for email field in profile settings.

* style: fix cycle layout height.
sriramveeraghanta pushed a commit that referenced this pull request Jan 22, 2024
* chore: add proper message for cycle/ module having start & end date but isn't active yet.

* fix: infinite loader after updating workspace settings.

* fix: user profile icon dropdown doesn't closes automatically.

* style: fix inconsistent padding in cycle empty state.

* chore: remove multiple `empty state` in labels settings and improve add label logic.

* style: fix inconsistent padding in project label, integration and estimates empty state.

* style: fix integrations settings breadcrumb title.

* style: add proper `disabled` styles for email field in profile settings.

* style: fix cycle layout height.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants