search

sparcopen / infrastructure

A home on the web for our community owned infrastructure work.
https://infrastructure.sparcopen.org
MIT License
4 stars 2 forks source link

consider a more elegant way to do sub sections in report view #83

Closed JosephMcArthur closed 4 years ago

JosephMcArthur commented 4 years ago

Currently reports that have multiple sub-sections, e.g 'research companies' here https://infrastructure.sparcopen.org/landscape-analysis look a bit messy.

@sssoz wondering if you'd have any ideas to address this that are very quick (~1-2 hours or so) to do from design to implementation (not including my time tagging various posts).

sssoz commented 4 years ago

@JosephMcArthur I think grouping them into a sub-section makes iterating over each post a lot more complicated. We could simply add a small header for each of these, above the title. It might look a bit repetitive?

Screenshot 2020-08-21 at 22 57 07
sssoz commented 4 years ago

Will try something tomorrow.

JosephMcArthur commented 4 years ago

I think what you have there is better than what I've done at far :)

sssoz commented 4 years ago

Just add section: [section title] in the frontmatter for posts that belong to a section. See https://github.com/sparcopen/infrastructure/commit/43db3456a2ac123a846f4e2394882a59645be35b

We could also add some logic so that the section title is only displayed once, at the top of a group of posts, but it might make the layout a lot messier (e.g. what would posts that don’t belong to any sections look like when displayed after a grouping?)

sssoz commented 4 years ago

There’s one example here https://test.infrastructure.sparcopen.org/landscape-analysis

JosephMcArthur commented 4 years ago

That's great, I'll just update the front-matter accordingly to propagate :) Thanks!

Joseph McArthur Assistant Director: Right to Research Coalition ( http://righttoresearch.org ) Director / Founder: Open Access Button ( http://openaccessbutton.org ) Twitter: @Mcarthur_Joe ( https://twitter.com/Mcarthur_Joe ) Skype: joseph_mcarthur

On Fri, Aug 21, 2020 at 11:48 PM, Sophy < notifications@github.com > wrote:

There’s one example here https:/ / infrastructure. sparcopen. org/ landscape-analysis ( https://infrastructure.sparcopen.org/landscape-analysis ) l

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub ( https://github.com/sparcopen/infrastructure/issues/83#issuecomment-678546870 ) , or unsubscribe ( https://github.com/notifications/unsubscribe-auth/ABFZMZ6KGIHOMSJUKZBRXMTSB32TZANCNFSM4ODI3NZA ).

JosephMcArthur commented 4 years ago

updated! It looks a lot better, thanks