Skip to content

fix: remove uneven padding from Repo button#3374

Merged
motorina0 merged 1 commit into
lnbits:devfrom
BenGWeeks:fix/repo-button-padding-issue-3371
Oct 2, 2025
Merged

fix: remove uneven padding from Repo button#3374
motorina0 merged 1 commit into
lnbits:devfrom
BenGWeeks:fix/repo-button-padding-issue-3371

Conversation

@BenGWeeks
Copy link
Copy Markdown
Contributor

@BenGWeeks BenGWeeks commented Oct 1, 2025

Summary

  • Fixes uneven padding on the Repo button in extension details dialog
  • Removed q-pr-xs class which was causing padding-right: 4px while left padding was 16px
  • Button now has balanced padding on both sides

Changes

  • lnbits/core/templates/core/extensions.html (line 919): Removed class="q-pr-xs" from Repo button

Screenshot

image

Test plan

  • Tested on local dev server (port 5001)
  • Verified button padding is now equal on left and right sides
  • Confirmed visual appearance matches other buttons in the UI

Fixes #3371

🤖 Generated with Claude Code

Removed `q-pr-xs` class from Repo button which was causing
padding-right: 4px while left padding was 16px, making the
button style look unbalanced.

Fixes lnbits#3371

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.10%. Comparing base (7c88702) to head (90121f5).
⚠️ Report is 1 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #3374      +/-   ##
==========================================
- Coverage   56.11%   56.10%   -0.01%     
==========================================
  Files         116      116              
  Lines       15090    15090              
==========================================
- Hits         8467     8466       -1     
- Misses       6623     6624       +1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@arcbtc
Copy link
Copy Markdown
Member

arcbtc commented Oct 2, 2025

Gosh, thank you! This always annoys me

@arcbtc arcbtc self-requested a review October 2, 2025 10:11
@motorina0 motorina0 merged commit 6c32ebf into lnbits:dev Oct 2, 2025
44 checks passed
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.

[BUG] Diverse UI for Repo and License on Allowance Extension

3 participants