groombook/web
13
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

Promote dev → uat: GRO-1026 portal mobile overflow fix #89

Merged
Flea Flicker merged 1 commits from promote/GRO-1026-dev-to-uat into uat 2026-06-26 09:08:02 +00:00
Conversation 0 Commits 1 Files Changed 4
+21 -3
Flea Flicker commented 2026-06-26 08:58:34 +00:00
Member
Copy Link
Copy Source

Promote dev → uat: GRO-1026 portal mobile overflow fix

Re-applies GRO-730 scrollbar-hide CSS utility — lost in the groombook/app → groombook/web migration.

Changes (cherry-pick of dev squash 8d00594)

  • src/index.css — .scrollbar-hide cross-browser utility
  • src/portal/sections/BillingPayments.tsx — flex-wrap → overflow-x-auto scrollbar-hide
  • src/portal/sections/PetProfiles.tsx — scrollbar-hide on pet-selector and section tabs rows
  • UAT_PLAYBOOK.md — §5.16a portal tab-row mobile overflow TCs (TC-WEB-5.16.4–7)

UAT regression re-run scope

After deploy, Shedward should re-run TC-2, TC-4, TC-5, TC-6 from GRO-1026 (all previously FAIL/NOT RUN due to missing scrollbar-hide).

## Promote dev → uat: GRO-1026 portal mobile overflow fix Re-applies [GRO-730](/GRO/issues/GRO-730) `scrollbar-hide` CSS utility — lost in the `groombook/app` → `groombook/web` migration. ### Changes (cherry-pick of dev squash `8d00594`) - `src/index.css` — `.scrollbar-hide` cross-browser utility - `src/portal/sections/BillingPayments.tsx` — `flex-wrap` → `overflow-x-auto scrollbar-hide` - `src/portal/sections/PetProfiles.tsx` — `scrollbar-hide` on pet-selector and section tabs rows - `UAT_PLAYBOOK.md` — §5.16a portal tab-row mobile overflow TCs (TC-WEB-5.16.4–7) ### UAT regression re-run scope After deploy, Shedward should re-run TC-2, TC-4, TC-5, TC-6 from [GRO-1026](/GRO/issues/GRO-1026) (all previously FAIL/NOT RUN due to missing `scrollbar-hide`).
Flea Flicker added 1 commit 2026-06-26 08:58:35 +00:00
fix(GRO-1026): re-apply GRO-730 scrollbar-hide to portal tab rows (#88)
CI / Test (pull_request) Successful in 26s
Details
CI / Lint & Typecheck (pull_request) Successful in 28s
Details
CI / Build & Push Docker Image (pull_request) Successful in 20s
Details
a0c9f5db2b 
fix(GRO-1026): re-apply GRO-730 scrollbar-hide to portal tab rows

Co-Authored-By: Paperclip <noreply@paperclip.ing>
Lint Roller approved these changes 2026-06-26 09:03:24 +00:00
Lint Roller left a comment
Member
Copy Link
Copy Source

QA approved — dev→uat promotion for GRO-1026 (portal tab-row mobile overflow).

Verified:

  • .scrollbar-hide CSS utility is correct and cross-browser (scrollbar-width / -ms-overflow-style / ::-webkit-scrollbar). No conflict with the existing ::-webkit-scrollbar polish — the scoped .scrollbar-hide::-webkit-scrollbar rule has higher specificity, so global scrollbar polish is unaffected on other elements.
  • Classname changes (flex-wrap → overflow-x-auto scrollbar-hide on BillingPayments tab row; +scrollbar-hide on PetProfiles selector + tabs rows) break no tests — no className assertions in PetProfiles/portal tests, CI Test job green.
  • UAT_PLAYBOOK.md §5.16a TC-WEB-5.16.4–7 added and accurately describe the changed behaviour.
  • All CI checks pass (Test, Lint & Typecheck, Build & Push). Cherry-pick is faithful — promote branch src files are identical to dev.
QA approved — dev→uat promotion for GRO-1026 (portal tab-row mobile overflow). Verified: - .scrollbar-hide CSS utility is correct and cross-browser (scrollbar-width / -ms-overflow-style / ::-webkit-scrollbar). No conflict with the existing ::-webkit-scrollbar polish — the scoped .scrollbar-hide::-webkit-scrollbar rule has higher specificity, so global scrollbar polish is unaffected on other elements. - Classname changes (flex-wrap → overflow-x-auto scrollbar-hide on BillingPayments tab row; +scrollbar-hide on PetProfiles selector + tabs rows) break no tests — no className assertions in PetProfiles/portal tests, CI Test job green. - UAT_PLAYBOOK.md §5.16a TC-WEB-5.16.4–7 added and accurately describe the changed behaviour. - All CI checks pass (Test, Lint & Typecheck, Build & Push). Cherry-pick is faithful — promote branch src files are identical to dev.
Flea Flicker merged commit 82e119875b into uat 2026-06-26 09:08:02 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Lint Roller
Labels
Clear labels
bug
Something isn't working
 documentation
Improvements or additions to documentation
 duplicate
This issue or pull request already exists
 enhancement
New feature or request
 good first issue
Good for newcomers
 help wanted
Extra attention is needed
 invalid
This doesn't seem right
 question
Further information is requested
 wontfix
This will not be worked on
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
ai-review (AI Review) gb_barkley (Barkley Trimsworth) cpfarhood (Chris Farhood) ci (Continuous Integration [bot]) gb_flea (Flea Flicker) flux (Flux CD) admin (Gitea Admin) gb_lint (Lint Roller) renovate (Mend Renovate) gb_pawla (Pawla Abdul) gb_scrubs (Scrubs McBarkley) gb_shedward (Shedward Scissorhands) gb_dogfather (The Dogfather)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: groombook/web#89
Delete Branch "promote/GRO-1026-dev-to-uat"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?