Skip to content
This repository has been archived by the owner on Dec 25, 2023. It is now read-only.

Reset/Leave Search #64

Closed
SerenaButler opened this issue Feb 25, 2019 · 7 comments
Closed

Reset/Leave Search #64

SerenaButler opened this issue Feb 25, 2019 · 7 comments
Labels
enhancement New feature or request

Comments

@SerenaButler
Copy link

Reset/Leave search (e.g. X in search box or Back-Button). There is no way resetting a search but deleting the search text.

@ildyria
Copy link
Member

ildyria commented Feb 25, 2019

ESC does not work ?
F5 should work too... 😃

@ildyria
Copy link
Member

ildyria commented Feb 25, 2019

Thank you for pointing all those. 👍

@SerenaButler
Copy link
Author

Oh F5 ... I didn't know that .... :-P

@SerenaButler
Copy link
Author

ESC is working but that is not good UX for a beginner. Recognition vs. Recall. ;-)

@ildyria ildyria transferred this issue from LycheeOrg/Lychee Apr 9, 2019
@ildyria ildyria added the enhancement New feature or request label Apr 9, 2019
@SerenaButler
Copy link
Author

I found out that a "x" is allready there. Haha. To see change e.g. user.css:

.header__clear {
opacity: 1;
}

I guess the idea was to just show the x in certain cases, e.g. allready searched.

@kamil4
Copy link
Contributor

kamil4 commented Apr 18, 2019

Yeah, that search results view needs to be cleaned up. I feel that it should be its own view (like album and photo views) but right now it's just a modified albums view which is confusing both to the user and to when coding the front end.

@kamil4
Copy link
Contributor

kamil4 commented Jun 15, 2019

This was fixed in #107, I believe.

@kamil4 kamil4 closed this as completed Jun 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants