- Merge request !282Issue #3315860: Change the position of the Browse tab in the local task order โ (Merged) created by srishtiiee
- ๐บ๐ธUnited States phenaproxima Massachusetts
Found a few relatively minor code quality things, and one bigger concern that I'm thinking can result in false positives if we don't correct it. Otherwise, looks straightforward to me.
- Open on Drupal.org โCore: 10.1.x + Environment: PHP 8.1 & MySQL 5.7last update
about 1 year ago Waiting for branch to pass - Open on Drupal.org โCore: 10.1.x + Environment: PHP 8.1 & MySQL 5.7last update
about 1 year ago Waiting for branch to pass - last update
about 1 year ago 78 pass - Status changed to Needs review
about 1 year ago 12:26pm 18 October 2023 - ๐บ๐ธUnited States chrisfromredfin Portland, Maine
I believe all the threads are resolved except for that huge one where I pasted the entire markup (RIP your scrollbar). Anyone want to take another look?
- Status changed to Needs work
8 months ago 3:22pm 2 April 2024 - ๐บ๐ธUnited States chrisfromredfin Portland, Maine
Following the usability study done with University of Minnesota, it became clear that browse really must be the FIRST tab, and therefore the default task for "Extend."
I've updated the proposed resolution. Marking "Needs Work"
- Status changed to Fixed
7 months ago 2:31pm 8 May 2024 - ๐บ๐ธUnited States chrisfromredfin Portland, Maine
Merged this because it gets us started and gets us a test, and we still need to discuss whether or not to make it the default tab (I believe so, but there is not consensus). There is another issue for that, so I like leaving this one.
Tests did fail, but they were separate fails on flaky FunctionalJavascript tests.
-
chrisfromredfin โ
committed ee68739b on 1.0.x authored by
srishtiiee โ
Issue #3315860 by chrisfromredfin, narendraR, srishtiiee, bnjmnm,...
-
chrisfromredfin โ
committed ee68739b on 1.0.x authored by
srishtiiee โ
Automatically closed - issue fixed for 2 weeks with no activity.