Skip to content

Fix photo ordering on albums - #4685

Merged
ildyria merged 4 commits into
masterfrom
narrower-left-menu
Aug 30, 2026
Merged

Fix photo ordering on albums #4685
ildyria merged 4 commits into
masterfrom
narrower-left-menu

Conversation

@ildyria

@ildyria ildyria commented Aug 30, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • New Features
    • Improved the photo details panel with responsive sidebar behavior, right-to-left layout support, and fullscreen display on smaller screens.
    • Enhanced the photo viewing layout and menu presentation for a more consistent experience.
  • Bug Fixes
    • Updated album sorting compatibility by standardizing affected sorting options.
  • Improvements
    • Refined star controls, menu sizing, and photo panel styling.
  • Version
    • Updated the application to version 7.8.2.

@ildyria
ildyria requested a review from a team as a code owner August 30, 2026 15:05
@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: b21ccedd-19db-4cd7-a95e-5ddf6bc0a90e

📥 Commits

Reviewing files that changed from the base of the PR and between 76dc307 and 4926803.

📒 Files selected for processing (1)
  • resources/js/v8/components/drawers/PhotoDetails.vue
🚧 Files skipped from review as they are similar to previous changes (1)
  • resources/js/v8/components/drawers/PhotoDetails.vue

Included review availability: Your plan provides up to 8 included reviews per hour; 4 remain after this review.


📝 Walkthrough

Walkthrough

The release normalizes album sorting values, updates the application version, and changes the photo interface to use shared sidebar state with RTL-aware responsive layout components.

Changes

Release and photo interface update

Layer / File(s) Summary
Release migrations and version metadata
database/migrations/*, version.md
Album sorting values are normalized. The stored version changes to 070802, cache clearing reports warnings, rollback restores 070801, and version.md changes to 7.8.2.
Shared photo details sidebar flow
resources/js/v8/components/drawers/PhotoDetails.vue, resources/js/v8/components/gallery/photoModule/PhotoPanel.vue
Photo details use USidebar and shared store state. The photo panel applies RTL-aware placement and dashboard layout components.
Photo interface shell adjustments
resources/js/v8/components/headers/PhotoHeader.vue, resources/js/v8/components/icons/StarRow.vue, resources/js/v8/menus/LeftMenu.vue
Header markers and non-null photo access are added. Star spacing changes, and left-menu content width is limited to max-w-xs.

Estimated code review effort: 3 (Moderate) | ~25 minutes

Merge Risk: 🔵 Low · up to 49268

The change is otherwise mergeable, but album rating icons may render incorrectly by stacking at one position; this bounded display issue should be corrected or explicitly accepted before merge.

Poem

A rabbit sees the sidebar glide,
With albums tucked neatly inside.
Sorting hops to “title” bright,
The cache clears softly in the night.
Seven-eight-two now marks the way.

🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed Docstring coverage is 80.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 5 functions across 2 files. (1 skipped: 1 u…
Full details: Docstring Coverage

Explanation

Docstring coverage is 80.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 5 functions across 2 files. (1 skipped: 1 unsupported.)


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

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot 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.

Actionable comments posted: 4

🧹 Nitpick comments (1)
resources/js/v8/components/headers/PhotoHeader.vue (1)

2-5: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

Remove the commented-out UDashboardNavbar wrapper.

Lines 2-5 and line 93 add commented markup that no longer participates in rendering. Delete both markers, or complete the migration to UDashboardNavbar.


ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 015b9b8a-b9f3-48c9-a1a5-3c0f1d70fcb8

📥 Commits

Reviewing files that changed from the base of the PR and between f16ed80 and a9ee8cc.

📒 Files selected for processing (8)
  • database/migrations/2026_08_30_150156_fix_sorting_photos.php
  • database/migrations/2026_08_30_150404_bump_version070802.php
  • resources/js/v8/components/drawers/PhotoDetails.vue
  • resources/js/v8/components/gallery/photoModule/PhotoPanel.vue
  • resources/js/v8/components/headers/PhotoHeader.vue
  • resources/js/v8/components/icons/StarRow.vue
  • resources/js/v8/menus/LeftMenu.vue
  • version.md

Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review.

Comment thread database/migrations/2026_08_30_150404_bump_version070802.php
Comment thread resources/js/v8/components/drawers/PhotoDetails.vue Outdated
Comment thread resources/js/v8/components/drawers/PhotoDetails.vue Outdated
Comment thread resources/js/v8/components/icons/StarRow.vue
@ildyria
ildyria merged commit 0831ec6 into master Aug 30, 2026
54 checks passed
@ildyria
ildyria deleted the narrower-left-menu branch August 30, 2026 17:03
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.

1 participant