Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(web): move search options into a modal #12438

Merged
merged 6 commits into from
Sep 10, 2024

Conversation

ben-basten
Copy link
Member

Description

Moving the search options into a modal, rather than displaying them below the search bar. Benefits of this change include:

  • Guaranteed space in all screen sizes for the filters
  • Focus trapping, so users can never accidentally tab off the end of the filters
  • Automatically return focus back to the button in the search bar when the modal is closed

Screenshots

Wide screen width

filter-modal-desktop

Mobile screen width

filter-modal-mobile

@ben-basten ben-basten changed the title feat(web): move search options into a modal, POC feat(web): move search options into a modal Sep 9, 2024
@ben-basten ben-basten marked this pull request as ready for review September 9, 2024 11:12
@alextran1502
Copy link
Contributor

Thank you for the PR. One change I think would make the model look better is incorporating the scroll bar inside the modal and appearing below the curved corner. Please see the example below

image

@alextran1502 alextran1502 enabled auto-merge (squash) September 10, 2024 04:00
@alextran1502 alextran1502 merged commit 02047a0 into main Sep 10, 2024
35 checks passed
@alextran1502 alextran1502 deleted the feat/search-options-modal branch September 10, 2024 04:12
spiceratops added a commit to spiceratops/k8s-gitops that referenced this pull request Sep 12, 2024
…115.0 (#964)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
|
[ghcr.io/immich-app/immich-server](https://redirect.github.com/immich-app/immich)
| minor | `v1.114.0` -> `v1.115.0` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>immich-app/immich (ghcr.io/immich-app/immich-server)</summary>

###
[`v1.115.0`](https://redirect.github.com/immich-app/immich/releases/tag/v1.115.0)

[Compare
Source](https://redirect.github.com/immich-app/immich/compare/v1.114.0...v1.115.0)

### v1.115.0

> \[!NOTE]
> We are working on redesigning the flows and layout of the mobile app
to incorporate recently added features such as folders and tags, as well
as making improvements to album management on the mobile app. We are
gathering feedback in this
[discussion](https://redirect.github.com/immich-app/immich/discussions/12597).
We are excited to hear your feedback.

Welcome to release `v1.115.0` of Immich. This release focuses on
cleaning up our backlog of issues. It is also worth noting that the
[Cursed knowledge page](http://immich.app/cursed-knowledge/) has been
updated with *even more cursed knowledge*. Please find some of the
release highlights below:

-   Administration button
-   Manually link and unlink Live motion photos
-   Default exclusion patterns
-   Start-up folder checks
-   Upload trash indicator

Notable bug fixes include:

-   Signing out of the web now signs out of all tabs
- Long-standing issue regarding inaccurate album asset count has been
fixed
- Search text is preserved when switching between "Context" and "File
name or extension"
-   Editing a person's name or birthdate on mobile has been fixed

##### Administration button

The `Administration` button on the web is now moved inside the profile
sheet to clean up the top app bar and improve the navigation experience
for all screen sizes.

<img width="346" alt="image"
src="https://github.com/user-attachments/assets/36389e3d-3fc9-4a4b-a480-f3165ec114bc">

##### Manually link and unlink Live motion photos
Live photos (iOS) can now be manually linked an unlinked.

[Screencast from 09-11-2024 10:15:57
AM.webm](https://redirect.github.com/user-attachments/assets/07ec144a-af03-4827-8e6b-f2b4f30ec1f4)

##### Default exclusion patterns

We have added default exclusion patterns whenever a new external library
is created. By default, we exclude the `@eaDir` folder Synology uses to
store miscellaneous files and any files beginning with `._`. Of course,
you can edit these patterns if you want to, but this is highly unlikely.


![image](https://redirect.github.com/user-attachments/assets/1ec7fe82-c0f0-4049-a3fa-54e42ebd5d19)

##### Startup folder checks

Immich will now *fail to start* if it is unable to read/write to
`upload/`, `library/`, `thumbs/`, `encoded-videos/` or `profile/`.
Starting with this release, the server will write an `.immich` file to
each of the previously listed folders. On subsequent restarts, it will
*fail to start* if it fails to locate the previously written `.immich`
file. This will hopefully increase transparency around situations where
the `UPLOAD_LOCATION` mount is incorrectly configured or unavailable.

##### Upload trash indicator

When you upload a duplicate via the web and that duplicate is in the
trash, you now will see an indicator for it, along with being able to
open the asset in a new tab.

| Light | Dark |
| - | - |
|
![image](https://redirect.github.com/user-attachments/assets/78facde1-db58-49bf-833e-e53179082308)
|
![image](https://redirect.github.com/user-attachments/assets/6a58bae3-0ccf-40dc-ad89-5b43da41faeb)
|

***

#### Support Immich

<p align="center">
<img
src="https://media.giphy.com/media/v1.Y2lkPTc5MGI3NjExbjY2eWc5Y2F0ZW56MmR4aWE0dDhzZXlidXRmYWZyajl1bWZidXZpcyZlcD12MV9pbnRlcm5hbF9naWZfYnlfaWQmY3Q9Zw/87CKDqErVfMqY/giphy.gif"
width="450" title="SUPPORT THE PROJECT!">
</p>

If you find the project helpful, you can support Immich by purchasing a
product key at <https://buy.immich.app>.

Cheers! 🍻

<!-- Release notes generated using configuration in .github/release.yml
at main -->

#### What's Changed

##### 🚀 Features

- feat(web): manually link live photos by
[@&#8203;jrasm91](https://redirect.github.com/jrasm91) in
[immich-app/immich#12514
- feat(web): unlink live photos by
[@&#8203;jrasm91](https://redirect.github.com/jrasm91) in
[immich-app/immich#12574

##### 🌟 Enhancements

- feat(web): logout of all tabs by
[@&#8203;jrasm91](https://redirect.github.com/jrasm91) in
[immich-app/immich#12407
- feat(server): start up folder checks by
[@&#8203;jrasm91](https://redirect.github.com/jrasm91) in
[immich-app/immich#12401
- feat(web): responsive top navigation by
[@&#8203;ben-basten](https://redirect.github.com/ben-basten) in
[immich-app/immich#12373
- feat(mobile): Add dismiss action on app_bar_dialog by
[@&#8203;Lauritz-Tieste](https://redirect.github.com/Lauritz-Tieste) in
[immich-app/immich#12511
- feat(web): select the EXIF timezone (if it exists) in dropdown by
[@&#8203;C-Otto](https://redirect.github.com/C-Otto) in
[immich-app/immich#12495
- fix(web): show trash indicator by
[@&#8203;jrasm91](https://redirect.github.com/jrasm91) in
[immich-app/immich#12521
- feat(web): move search options into a modal by
[@&#8203;ben-basten](https://redirect.github.com/ben-basten) in
[immich-app/immich#12438
- feat(server): default exclusion patterns by
[@&#8203;etnoy](https://redirect.github.com/etnoy) in
[immich-app/immich#12566

##### 🐛 Bug fixes

- fix(server): remove hidden assets from albums by
[@&#8203;jrasm91](https://redirect.github.com/jrasm91) in
[immich-app/immich#12449
- fix(server): handle multiple hierarchical subjects by
[@&#8203;jrasm91](https://redirect.github.com/jrasm91) in
[immich-app/immich#12509
- fix(mobile): scroll thumb is hidden behind the tab navigation bar by
[@&#8203;alextran1502](https://redirect.github.com/alextran1502) in
[immich-app/immich#12512
- fix(web): select partner assets from timeline by
[@&#8203;jrasm91](https://redirect.github.com/jrasm91) in
[immich-app/immich#12517
- fix(web): preserve search text by
[@&#8203;jrasm91](https://redirect.github.com/jrasm91) in
[immich-app/immich#12531
- fix(web): show upload error message on network error by
[@&#8203;jrasm91](https://redirect.github.com/jrasm91) in
[immich-app/immich#12533
- fix(server): handle invalid directory item by
[@&#8203;jrasm91](https://redirect.github.com/jrasm91) in
[immich-app/immich#12534
- fix(web): leave assets selected after add to album by
[@&#8203;jrasm91](https://redirect.github.com/jrasm91) in
[immich-app/immich#12537
- fix(server): person repo methods by
[@&#8203;jrasm91](https://redirect.github.com/jrasm91) in
[immich-app/immich#12524
- fix(ml): batch axis not being added for recognition model by
[@&#8203;mertalev](https://redirect.github.com/mertalev) in
[immich-app/immich#12588

##### 📚 Documentation

- docs: more cursed knowledge by
[@&#8203;jrasm91](https://redirect.github.com/jrasm91) in
[immich-app/immich#12529
- docs: proper value of word-based suggestions setting to setup VSCode
by [@&#8203;sapachev](https://redirect.github.com/sapachev) in
[immich-app/immich#12586
- docs: add server backup to First Steps by
[@&#8203;mmomjian](https://redirect.github.com/mmomjian) in
[immich-app/immich#12555

##### 🌐 Translations

- chore(web): update translations by
[@&#8203;weblate](https://redirect.github.com/weblate) in
[immich-app/immich#12384

#### New Contributors

- [@&#8203;pbustamantes](https://redirect.github.com/pbustamantes) made
their first contribution in
[immich-app/immich#12486
- [@&#8203;Lauritz-Tieste](https://redirect.github.com/Lauritz-Tieste)
made their first contribution in
[immich-app/immich#12511
- [@&#8203;jonathanjsimon](https://redirect.github.com/jonathanjsimon)
made their first contribution in
[immich-app/immich#12510
- [@&#8203;sapachev](https://redirect.github.com/sapachev) made their
first contribution in
[immich-app/immich#12586

**Full Changelog**:
https://github.com/immich-app/immich/compare/v1.114.0...

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://redirect.github.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC43Ny4yIiwidXBkYXRlZEluVmVyIjoiMzguNzcuMiIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsicmVub3ZhdGUvY29udGFpbmVyIiwidHlwZS9taW5vciJdfQ==-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants