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

task/WP-65-DropdownViewFullPath - dropdown-menu css component #875

Conversation

wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented Oct 4, 2023

Overview

Do not duplicate dropdown-menu styles (to resolve #866 (comment)).

Related

Changes

  • added dropdown-menu.css
  • changed Data Files Breadcrumbs & Sidebar to use new CSS component
  • changed Data Files Breadcrumbs to use temporary ID1 for specificity

Testing

  1. Open https://cep.test/workbench/data/.
  2. Navigate to a deeply-nested folder.
  3. Open [ + Add ] and [ Go to ... ] menus.
  4. Verify UI matches task/WP-65-DropdownViewFullPath #866.
  5. If UI differs, verify UI matches design.

UI

add go to ...
DataFilesSidebar DataFilesBreadcrumbs

Notes

There are some undesirable UI hacks and bugs. They may be fixed in separate PRs, because they seem to be caused by issues from target branch or main branch. If actually this PR is causing new bugs, please report.

Footnotes

  1. Use of ID for this is not recommended, hence "temporary". See UI - CSS Style Guide - Specificity @ IDs in CSS.

@codecov
Copy link

codecov bot commented Oct 4, 2023

Codecov Report

Merging #875 (86afa44) into task/WP-65-DropdownViewFullPath (d20f4cb) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@                       Coverage Diff                        @@
##           task/WP-65-DropdownViewFullPath     #875   +/-   ##
================================================================
  Coverage                            62.94%   62.94%           
================================================================
  Files                                  427      427           
  Lines                                12219    12219           
  Branches                              2513     2513           
================================================================
  Hits                                  7691     7691           
  Misses                                4319     4319           
  Partials                               209      209           
Flag Coverage Δ
javascript 68.56% <ø> (ø)
unittests 57.29% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
...iles/DataFilesBreadcrumbs/DataFilesBreadcrumbs.jsx 72.52% <ø> (ø)

@wesleyboar
Copy link
Member Author

@wesleyboar wesleyboar merged commit a72b22c into task/WP-65-DropdownViewFullPath Oct 4, 2023
6 checks passed
@wesleyboar wesleyboar deleted the task/WP-65-DropdownViewFullPath--dropdown-menu-css-component branch October 4, 2023 22:19
chandra-tacc added a commit that referenced this pull request Oct 27, 2023
* task/WP-65-DropdownViewFullPath

* task/WP-65-DropdownViewFullPath-v2

* task/WP-65-DropdownViewFullPath-v3

* task/WP-65-DropdownViewFullPath-v4

* task/WP-65-DropdownViewFullPath-v5

* task/WP-65-DropdownViewFullPathv6

* task/WP-65-DropdownViewFullPath-v7

* task/WP-65-DropdownViewFullPath - dropdown-menu css component (#875)

* task/WP-65-DropdownViewFullPath-v8

* task/WP-65-DropdownViewFullPath-v9

* task/WP-65-DropdownViewFullPath-v10

* task/WP-65-DropdownViewFullPath-v11

* task/WP-65-DropdownViewFullPath-v12

* task/WP-65-DropdownViewFullPath-v13

* task/WP-65-DropdownViewFullPath-v14

* task/WP-65-DropdownViewFullPath-v15

* Update client/src/components/DataFiles/DataFilesModals/DataFilesShowPathModal.jsx

Co-authored-by: Sal Tijerina <[email protected]>

* prettier fix

---------

Co-authored-by: Taylor Grafft <[email protected]>
Co-authored-by: Taylor Grafft <[email protected]>
Co-authored-by: Sal Tijerina <[email protected]>
Co-authored-by: Wesley B <[email protected]>
Co-authored-by: Taylor Grafft <[email protected]>
Co-authored-by: Taylor Grafft <[email protected]>
Co-authored-by: Chandra Y <[email protected]>
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.

1 participant