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

#3024 [liqd.net] Liquid Academy Content Detail Page #7

Open
mcastro-lqd opened this issue Jun 24, 2020 · 3 comments
Open

#3024 [liqd.net] Liquid Academy Content Detail Page #7

mcastro-lqd opened this issue Jun 24, 2020 · 3 comments

Comments

@mcastro-lqd
Copy link

mcastro-lqd commented Jun 24, 2020

https://stage.liqd.net/en/liquid-academy/was-ist-liquid-academy/

Sorry if I wasn't so clear with the mockups, but here are some comments regarding the Content Detail Pages. Let me know if you have any questions :)

01 – As mentioned on the provided CI I thought about making the blog articles Body paragraphs using 16px, so we could have more contrast with the 19,2px one and play around with the requested new Subheads. Actually, both Desktop and Mobile are a bit different than on Zeplin.
Screenshot-2020-06-24-at-14 28 50

02 – Can we make the categories here clickable? So the users would be able to go back to the overview screen already filtered with this category.
Screenshot 2020-06-24 at 14 32 03

03 – Here as well (that's why I left underlined, so it means when hovered its clicable)
Screenshot 2020-06-24 at 16 07 51

04 – Spacing here is also a bit off, maybe we could make the headlines and dates a bit less separated and make the rows a bit more separated to make the division clearer, do you know what I mean? On Zeplin is easier to understand.
Screenshot 2020-06-24 at 14 28 31

05 – And the arrow on Mobile is a bit weird.
IMG_0005

Thanks in advance!

@philli-m
Copy link
Contributor

philli-m commented Jul 1, 2020

@mcastro-lqd danke!
02 / 03 - this is another case of enhancement as we didn't poker with that in mind so I will add an enhancement label and if @rittermo is up for it, we can make an enhancement story in next iteration but for now I can make them link to overview page if you want? maybe for future we should look at ways to communicate not totally obvious behaviors, like adding a comment with page name to say if an element should link?
05 - what mobile are you using as can't reproduce

@philli-m philli-m self-assigned this Jul 1, 2020
@philli-m philli-m removed their assignment Oct 28, 2020
@lthueer
Copy link

lthueer commented Nov 10, 2020

@phillimorland are the issues 1 and 4 done? or all of the above open?

@philli-m
Copy link
Contributor

@lthueer I did these fixes in July so i'm not sure but from comment 2 and 3 are not done as it would count as an enhancement and 5 if i couldn't reproduce I can't fix it so no not all done

@goapunk goapunk transferred this issue from another repository Aug 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants