Skip to content

PICARD-3234: Fix columns being restored with 0 width#3099

Merged
zas merged 1 commit intometabrainz:masterfrom
phw:PICARD-3234-fix-add-columns
Mar 19, 2026
Merged

PICARD-3234: Fix columns being restored with 0 width#3099
zas merged 1 commit intometabrainz:masterfrom
phw:PICARD-3234-fix-add-columns

Conversation

@phw
Copy link
Member

@phw phw commented Mar 19, 2026

Summary

  • This is a…
    • Bug fix
    • Feature addition
    • Refactoring
    • Minor / simple change (like a typo)
    • Other
  • Describe this change in 1-2 sentences:

Problem

When loading the stored columns setting invisible columns would be set to a width of 0. This would call issues with columns not being shown after enabling them.

Solution

Set the width to the default column width if it would be 0.

AI Usage

In accordance with the AI use policy portion of the MetaBrainz Contribution Guidelines, the level of AI/LLM use in the development of this Pull Request is:

  • No AI/LLM use
  • Minimal use (e.g. autocompletion)
  • Moderate use (e.g. suggestions regarding code fragments)
  • Significant use (e.g. code structure, tests development, etc.)
  • Primarily AI developed
  • Other (please specify below)

Action

Additional actions required:

  • Update Picard documentation (please include a reference to this PR)
  • Other (please specify below)

When loading the stored columns setting invisible columns would be set
to a width of 0. This would call issues with columns not being shown
after enabling them.

Set the width to the default column width if it would be 0.
Copy link
Collaborator

@zas zas left a comment

Choose a reason for hiding this comment

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

LGTM

@zas zas merged commit 1486bfd into metabrainz:master Mar 19, 2026
51 checks passed
@phw phw deleted the PICARD-3234-fix-add-columns branch March 20, 2026 07:08
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.

2 participants