fix(qt): reseat quorum labels when new types are inserted

[UdjinM6]

Issue being fixed or feature implemented

When new quorum types appear during reindexing, existing labels that follow the insertion point in the grid were not being repositioned, causing overlapping text in the Information tab's Quorums section.

before:

after:

What was done?

How Has This Been Tested?

reindex with and without the patch on testnet

Breaking Changes

n/a

Checklist:

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have added or updated relevant unit/integration/functional/e2e tests
• I have made corresponding changes to the documentation
• I have assigned this pull request to a milestone

[github-actions]

✅ No Merge Conflicts Detected

This PR currently has no conflicts with other open PRs.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2e6a225 and 641b3ea.

📒 Files selected for processing (1)

• src/qt/networkwidget.cpp

---

Walkthrough

The change modifies NetworkWidget::handleQrDataChanged to implement a two-stage widget reseating mechanism. When a new quorum label is inserted, the code sets a flag indicating reseating is required and immediately adds the new widgets to the grid. Subsequently, if entries were removed or displaced, the code repositions existing widgets by removing and re-adding them to ensure proper alignment without gaps.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title directly describes the main fix: reseating quorum labels when new types are inserted, which matches the changeset's core purpose of preventing label overlap.
Description check	✅ Passed	The description clearly explains the issue (overlapping labels when new quorum types appear), shows before/after screenshots, and details testing approach, all directly related to the changeset.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[kwvg]

utACK 641b3ea