Skip to content

Add gap between drafted books - #4099

Merged
Nateowami merged 1 commit into
masterfrom
fix/draft-gap
Sep 11, 2026
Merged

Nateowami merged 1 commit into
masterfrom
fix/draft-gap

Conversation

@RaymondLuong3

@RaymondLuong3 RaymondLuong3 commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

This adds the gap between books in a draft build. This was a regression from PR #4052

Before
Draft books before

After
Draft books after


This change is Reviewable

@codecov

codecov Bot commented Sep 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.94%. Comparing base (c607c80) to head (441da56).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4099   +/-   ##
=======================================
  Coverage   80.94%   80.94%           
=======================================
  Files         664      664           
  Lines       42153    42153           
  Branches     6944     6944           
=======================================
  Hits        34121    34121           
  Misses       6867     6867           
  Partials     1165     1165           

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

@RaymondLuong3
RaymondLuong3 deployed to screenshot_diff September 10, 2026 21:16 — with GitHub Actions Active

@Nateowami Nateowami 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.

:lgtm:

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


src/SIL.XForge.Scripture/ClientApp/src/app/translate/draft-generation/draft-preview-books/draft-preview-books.component.scss line 4 at r2 (raw file):

  display: flex;
  gap: 8px;
  flex-wrap: wrap;

I just added flex-wrap: wrap; because it's also needed and was also dropped.

@Nateowami
Nateowami enabled auto-merge (squash) September 11, 2026 01:45
@Nateowami
Nateowami deployed to screenshot_diff September 11, 2026 01:52 — with GitHub Actions Active
@Nateowami
Nateowami merged commit e0ff910 into master Sep 11, 2026
29 checks passed
@Nateowami
Nateowami deleted the fix/draft-gap branch September 11, 2026 01:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants