Skip to content

SF-3887 Fix layout of generate draft summary page on mobile - #4048

Merged
RaymondLuong3 merged 1 commit into
masterfrom
fix/SF-3887-mobile-generate-draft-fix
Aug 19, 2026
Merged

SF-3887 Fix layout of generate draft summary page on mobile#4048
RaymondLuong3 merged 1 commit into
masterfrom
fix/SF-3887-mobile-generate-draft-fix

Conversation

@Nateowami

@Nateowami Nateowami commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

(both screenshots have the bottom truncated because I screenshotted a node that wasn't fully in the viewport, but you can see the layout difference).

Before After
summary_page_before summary_page_after

This change is Reviewable

@codecov

codecov Bot commented Aug 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.10%. Comparing base (f7468c8) to head (94a46f2).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4048   +/-   ##
=======================================
  Coverage   81.10%   81.10%           
=======================================
  Files         661      661           
  Lines       43060    43060           
  Branches     7054     7059    +5     
=======================================
  Hits        34925    34925           
  Misses       6962     6962           
  Partials     1173     1173           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@RaymondLuong3 RaymondLuong3 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the useful descriptions

@RaymondLuong3 reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on Nateowami).

@RaymondLuong3 RaymondLuong3 self-assigned this Aug 17, 2026
@Nateowami Nateowami added testing complete Testing of PR is complete and should no longer hold up merging of the PR and removed ready to test labels Aug 19, 2026
@Nateowami
Nateowami force-pushed the fix/SF-3887-mobile-generate-draft-fix branch from c9e91bb to 590f4fd Compare August 19, 2026 14:10
@Nateowami
Nateowami deployed to screenshot_diff August 19, 2026 14:18 — with GitHub Actions Active
@RaymondLuong3
RaymondLuong3 force-pushed the fix/SF-3887-mobile-generate-draft-fix branch from 590f4fd to 94a46f2 Compare August 19, 2026 17:55
@RaymondLuong3
RaymondLuong3 enabled auto-merge (squash) August 19, 2026 18:10
@RaymondLuong3
RaymondLuong3 deployed to screenshot_diff August 19, 2026 19:15 — with GitHub Actions Active
@github-actions

Copy link
Copy Markdown
Contributor

📸 Screenshot diff deployed! (1 change)

View the visual diff at: https://pr-4048--sf-screenshot-diffs.netlify.app

@RaymondLuong3
RaymondLuong3 merged commit fc2eb4b into master Aug 19, 2026
27 of 28 checks passed
@RaymondLuong3
RaymondLuong3 deleted the fix/SF-3887-mobile-generate-draft-fix branch August 19, 2026 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing complete Testing of PR is complete and should no longer hold up merging of the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants