Skip to content

Keep filter panel open while selecting, add explicit close button - #10

Closed
Ken709-mp4 wants to merge 6 commits into
sclass53:mainfrom
Ken709-mp4:main
Closed

Ken709-mp4 wants to merge 6 commits into
sclass53:mainfrom
Ken709-mp4:main

Conversation

@Ken709-mp4

Copy link
Copy Markdown
Contributor

PR: Keep filter panel open while selecting, add explicit close button

Problem

The filter dropdown panel (#color-filter) felt like it closed after a single selection, making it cumbersome to pick multiple ratings, colors, lenses, or focal values in one go.

Changes

  1. Explicit close button

    • Added a small ✕ button in the top-right corner of #color-filter.
    • Clicking it is now the intended way to close the panel.
  2. Panel stays open during multi-select

    • Added e.stopPropagation() on the panel itself, so clicks on ratings, colors, lenses, or focal inputs no longer bubble up and close the dropdown accidentally.
  3. Cleanup

    • Fixed clearSharedFilters() to reset the checkbox-based rating filter correctly (it was trying to set .value on the checkbox container).
    • Removed the stale ratingFilterEl.value = String(minRating) sync from the rejects view rating dropdown.
  4. i18n

    • Added photos.filterPanelClose key to zh-CN.json, en-US.json, and messages.js.

Files changed

  • src/index.html
  • src/styles.css
  • src/app.js
  • src/locales/zh-CN.json
  • src/locales/en-US.json
  • src/locales/messages.js

Verification

  • npm run tauri dev compiled and opened the app window.
  • No frontend JS/CSS syntax errors.
  • node -c src/app.js passed.

你的GitHub用户名 added 6 commits August 30, 2026 15:31
… panels, custom menu bg color

- Move star-rating filter (0–5) from searchbar into #color-filter dropdown panel.
- Apply shared .glass liquid-glass style to color-filter, dialogs, preview panel, and selection bar.
- Add Liquid Glass theme in Settings alongside Dark/Light.
- Add custom menu background color picker, persisted via Tauri set_setting/get_setting.
- Update zh-CN / en-US / messages locales.
…lose ×

- Add a small ✕ close button to #color-filter so users can make multiple selections without the panel closing.\n- Stop click events inside the panel from bubbling, preventing accidental close.\n- Fix clearSharedFilters to reset checkbox-based rating filter instead of setting .value on a <div>.\n- Update i18n (zh-CN/en-US/messages.js) with filterPanelClose.
… close ×

- Replace flat backgrounds with radial gradient and glassmorphism across sidebar, searchbar, cards, toolbars, statusbar.\n- Add hover/active transitions and scale effects to buttons and cards.\n- Add fadeIn/scaleIn entry animations for cards, dialogs, preview, and filter panel.\n- Remove the explicit ✕ close button from #color-filter.\n- Keep the existing toggle behavior: click 筛选 again to close the dropdown.\n- Clean up filterPanelClose i18n key and related listeners.
- Replace <select> dropdown for semantic/tag search with two pill buttons matching the filter panel style.\n- Add horizontal padding to .view so main content no longer stretches edge-to-edge.\n- Keep searchbar, toolbar, and statusbar spanning full width inside padded views.\n- Fix duplicate semantic-search-input id in HTML.
…boxes

- Add left margin and max-width to .settings so the frame no longer hugs the left edge.\n- Make the edit-tag/confirm/rate dialogs more transparent (lighter glass).\n- Replace the rejects view rating dropdown with checkbox set inside #reject-cond-panel, matching the photos filter panel.\n- Share activeRatings Set between photos and rejects; update clear/reset logic.
renderFilterDots, renderRejectConds and renderRejectRatings run at module-load time with the default en-US locale. After initI18n() resolves the saved language and applyStaticI18n() updates static text, re-render these dynamic panels so the saved locale is reflected on first paint.
@Ken709-mp4 Ken709-mp4 closed this Aug 30, 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.

1 participant