Princeton-CDH / startwords

a research periodical irregularly published by the CDH.
https://startwords.cdh.princeton.edu
3 stars 3 forks source link

As a reader, I want to read a statement from the editor describing the project. #18

Closed thatbudakguy closed 4 years ago

thatbudakguy commented 4 years ago

dev notes

this is the about page, and it includes the masthead as specified in #21.

this issue also includes the default page summary template, because we want to show a preview of (and link to) the about page on the homepage. no longer using default summary since we want to control how it displays on the homepage.

rlskoeser commented 4 years ago

Grant and I discussed — masthead will be included on the project about page because we want to keep things simple and reduce the number of clicks and pages.

The about page already exists on the current site as a standard hugo page, under content/about.md

thatbudakguy commented 4 years ago

making this a 1 so we can use it to test the default summary.html template - once the about page is done, we will want to test that its summary renders correctly on the homepage, so that people can get to the editorial statement/masthead/etc, and this seems like most relevant place to test that.

edit: we aren't using the default summary, since we need to control exactly what displays on the homepage.

thatbudakguy commented 4 years ago

ok, this is ready for :eyes: from both @gwijthoff and @gissoo.

gissoo commented 4 years ago

@thatbudakguy This great,

May be irrelevant to you Nick, so tagging more people @gwijthoff @rlskoeser

rlskoeser commented 4 years ago

I also wondered about the | !

Would a definition list make sense here?

(Although I think these are content questions not related to the page styles)

gwijthoff commented 4 years ago

Just changed your title, @gissoo!

The |'s are wholly my quirk. I love |'s. Please feel free to suggest another solution!

As for "Masthead," my thinking in using that section heading is that it signifies something about the hybrid nature of this publication. Academic journals tend to have "editorial boards," while periodicals and newspapers have "mastheads." It suggests that the writing here is built for public audiences. But I'm open to talking more about it. Thoughts @thatbudakguy @rlskoeser? If we decide we like the term "masthead," maybe the layout/design of the section heading should just change? Maybe h2 rather than h1, or insert some extra space between the end of the "about" text and the new heading?

Also, seconding @gissoo's comment on the indentation of this page's header. It's moved a little to the right on mobile, just not quite far enough.

IMG_FFD6CDBD2ECC-1

thatbudakguy commented 4 years ago

there is the same issue with the page title "About" and the indented section at the top, where they should each shift one column to the right to match the design of the article page for mobile and tablet.

shifting the content in this page's header is a little more tricky, because this page - like all non-article pages - doesn't have a concept of a "header". for the time being, I can just add some html directly into the markdown to control it, but wanted to bring it up since it's not clear what a "header" would be in the context of a non-article page. if we wanted to edit the basic page template to include a header, we could do that, but then what would go in the header? just the title? we have no way of knowing beforehand what content the page would include, so it's kind of a thorny question.

I'm fine either way about the |, but it seems like if we do it, shouldn't RSK and I both be listed in a list under one |, like the board of advisors is?

gwijthoff commented 4 years ago

@thatbudakguy @rlskoeser up to you both how you'd like to be described / listed!

As for the "header" question, I don't have any real preference for how it's implemented, so long as "About" and the "startword" dictionary definition aren't covered by the nav icon when initially loaded on mobile.

gissoo commented 4 years ago

thatbudakguy commented 4 years ago

ok, I've added a solution that I think will work going forward.

in the future, if a new (non-article) page is made, all of the content that goes in its "header" will have the correct margins, so long as that content is stored in the YAML front matter summary for the page.

@gissoo let me know if the margin scaling looks right to you (check desktop as well). @gwijthoff let me know if the above description doesn't make sense. it's a little bit of an abuse of the "summary" function, but I think if we're aware of it it will work OK.

gissoo commented 4 years ago

@thatbudakguy thank you for your work on this and for coming up with this solution!! This is great!! The header's margins are incorrect on desktop, they should align with the body. They look perfect on mobile.

gwijthoff commented 4 years ago

@thatbudakguy I like this solution. Agreed with @gissoo that the header's margins look great on mobile now, but that they're too roomy on desktop. It looks awkward with how far over to the right "About" and the Startwords dictionary definition are placed.

thatbudakguy commented 4 years ago

ok, updated default pages to have the same header margins as article pages.

gissoo commented 4 years ago

@thatbudakguy thank you so much!! This looks great!