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

Fix inconsistent studio dashboard layout #964

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

adriangohjw
Copy link
Contributor

@adriangohjw adriangohjw commented Dec 27, 2024

Problem

currently the dashboard layout for "root page", "folder" and "collection" are all incorrect, which leads to a mild inconsistencies when navigating throughtout the pages

it also means logic are duplicated and fixes have to be checked against every part

this also fixes the double scrollbar issue introduced in #959

Solution

Breaking Changes

  • Yes - this PR contains breaking changes
    • Details ...
  • No - this PR is backwards compatible

Improvements:

  • refactor to use common Dashboard component

Before & After Screenshots

BEFORE:

Screen.Recording.2024-12-27.at.11.50.06.PM.mov

AFTER:

Screen.Recording.2024-12-27.at.11.51.39.PM.mov

Tests

  • navigate between the root page, folder and collection -> the layout should be consistent

@adriangohjw adriangohjw added bug Something isn't working enhancement New feature or request labels Dec 27, 2024
@adriangohjw adriangohjw self-assigned this Dec 27, 2024
@adriangohjw adriangohjw requested a review from a team as a code owner December 27, 2024 15:53
@datadog-opengovsg
Copy link

datadog-opengovsg bot commented Dec 27, 2024

Datadog Report

Branch report: fix-inconsistent-studio-dashboard-layout
Commit report: 9287b36
Test service: isomer-studio

✅ 0 Failed, 245 Passed, 36 Skipped, 47.07s Total Time
➡️ Test Sessions change in coverage: 1 no change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant