-
Notifications
You must be signed in to change notification settings - Fork 0
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
Gallery #47
Draft
TLTimeplex
wants to merge
9
commits into
main
Choose a base branch
from
gallery
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Gallery #47
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
+ Added gallery path to project
* Renamed page component to a more clear name + Added default route (May change later to QUERY) + Added custom db query to connect fursuits with the score + Added page separation for splitting the gallery + Added image details on click to show further information (currently only the score + For the "full" image view the user can now click in the detailed view on the image to resolve in the image_url * Modified responsibility to help mobile (Still wonky)
Open
+ Pulled out Pagnation component * Put all related components into their own folder + Added ts interface for Fursuits
* Fixed typo
* Changed page change procedure to hold on the search term + Added Search bar + Added Counter for amount of fursuits in gallery (Currently counting only displayable) + Added the button for sorting (Not implemented yet ^^) - Removed unused logic in controller + Added db query to get the highest ranking users (Top 3)
+ Added route to get the amount of fursuits available in the gallery (All) >> Old Counter is now used to show the amount of fursuits that match the search requirements, displayed as {ALL (SEARCH)} currently + Added functionality, that the search term now gets split by ' ' and then AND connected. (May also add a || to add an OR search later) + View added a per session request to get the total amount of fursuits and not to request the value on any request * Fixed sorting button to be a button and no longer submit
+ Automatically generating webp version of image if not present yet
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Gallery to view over all of the fursuit images
Linked to #45
Features (currently):
To-Dos
Further enhancments
Images (New->Old)
Search for Name
Current State (outdated)
Clear view and details
Click again on Image to get the image itself