Skip to content

Reduce gallery preview card width by 20% - #305

Merged
TheWitness merged 2 commits into
Cacti:developfrom
alcatron:fix/gallery-card-size-20261012
Oct 12, 2026
Merged

TheWitness merged 2 commits into
Cacti:developfrom
alcatron:fix/gallery-card-size-20261012

Conversation

@alcatron

Copy link
Copy Markdown
Contributor

Reduce gallery preview card width from 480px to 384px (20%) for a more compact map gallery. Generated thumbnail resolution, image aspect ratios and the responsive maximum width remain unchanged.

Validation: tried and approved on a running Cacti installation. Both existing gallery preview regression cases pass; manifest and whitespace checks pass. Includes a develop changelog entry.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The focused CSS change matches the stated requirement and introduces no unresolved issues.

0 open findings

What changed in this PR

Reduces gallery preview card width by 20% for a more compact layout while preserving thumbnail behavior.

Changes:

  • Changes card width from 480px to 384px.
  • Adds a develop changelog entry.
File Description
css/​weathermap.css Reduces gallery card width while retaining responsive sizing.
CHANGELOG.md Documents the gallery sizing change.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The focused CSS change matches the stated requirements and introduces no unresolved issues.

0 open findings

🧠 Review effort: Balanced

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The focused CSS change matches the stated requirements without introducing unresolved issues.

0 open findings

🧠 Review effort: Balanced

TheWitness
TheWitness previously approved these changes Oct 12, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The focused CSS change matches the stated requirement without altering responsiveness or image sizing.

0 open findings

🧠 Review effort: Balanced

@TheWitness
TheWitness merged commit ae5a9a1 into Cacti:develop Oct 12, 2026
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.

3 participants