-
Notifications
You must be signed in to change notification settings - Fork 17
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(directory): Add front page categories #4560
feat(directory): Add front page categories #4560
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Het gaat de goede richting op, maar er is iets waar ik niet helemaal mijn vinger op kan leggen wat waardoor het er nog niet optimaal uitziet. Ik denk dat het komt omdat de afbeeldingen en de tekst behoorlijk groot zijn in vergelijking tot het voorbeeld in het ontwerp document. Ook is de ruimte tussen de afbeeldingen best groot.
Als ik de grootte van het window aanpas dan komen de afbeeldingen soms over elkaar heen te staan.
… of github.com:molgenis/molgenis-emx2 into feat/3769-directory-update-frontpage-to-use-categories
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
_Functioneel werkt het, maar visueel is het nog behoorlijk dominant en zwaar.
- Ik denk dat het mooier is als er minder nadrukkelijk een rand om de afbeeldingen zijn en er wat meer afbeeldingen op een regel zodat de afstand tussen twee afbeeldingen kleiner is.
- Ook de tekst erboven mag wat minder nadruk hebben, een h3 ipv h1 vind ik zelf al mooier.
- BBMRI-ERIC is volledig in hoofdletters
- De tekst "Search the BBMRI-ERIC Directory by biobank, samples and collections" is incorrect, dat zou moeten zijn "Search the BBMRI-ERIC Directory by category"
Ik heb even wat met de web inspector gespeeld om de CSS on the fly aan te passen, dit is nog niet helemaal goed, maar geeft het idee wel weer:
Hiervoor moest ik met name de percentages in de flex en max-width aanpassen zodat er meer op een regel komt. Daarnaast de border en shadow aangepast. Dat is natuurlijk bestaande CSS classes aanpassen ipv de juiste alternatieve CSS classes kiezen, dus is niet 1 op 1 over te nemen._
ik denk niet dat dit beter is
suggested changes are no an improvement
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found some issues:
- After clicking on one of the filters, the history gets messed up and I need to press back 3 times to get back to the home screen.
- When going back, the query stays in the url, it should be removed when on the home screen.
- Clicking on 'see more detail' for a collection results in a white screen.
- Clicking the 'go to biobank' button results in a white screen.
- There are 4 collections previewed, but there are only 3 in the tables. Should this be dynamic?
is this related to this pr ? , do you have any suggestion on how to solve this ( i checked , imo this is related to the way the filters are loaded and the bookmark and route keeps getting updated, we should fix this ( not a small fix ) but not in this pr imo, this pr does make this issue more prominent, so maybe we should fix it before adding this feature) see: #4572
is this related to this pr ? , do you have any suggestion on how to solve this ( i checked , imo this is related to the way the filters are loaded and the bookmark and route keeps getting updated, we should fix this ( not a small fix ) but not in this pr imo, this pr does make this issue more prominent, so maybe we should fix it before adding this feature) see: #4572
can not reproduce
not sure what you mean by this , and how it relates to this pr |
…3769-directory-update-frontpage-to-use-categories
- tweak style, make it a bit more subtle
Quality Gate passedIssues Measures |
What are the main changes you did
note: to test, enable frontpage by setting initialLandingpage.enabled to true
Todo
link to config, tie to or extend some/all stuff to config( can be separate pr if needed )How to test
Checklist