Skip to content

Update images for about page, refresh the layout#1960

Open
Mattsi-Jansky wants to merge 16 commits intoemfcamp:mainfrom
Mattsi-Jansky:about-page-2026
Open

Update images for about page, refresh the layout#1960
Mattsi-Jansky wants to merge 16 commits intoemfcamp:mainfrom
Mattsi-Jansky:about-page-2026

Conversation

@Mattsi-Jansky
Copy link
Copy Markdown
Contributor

@Mattsi-Jansky Mattsi-Jansky commented Apr 9, 2026

image

All images comply with photo policy, by my judgement at least. Welcome to review and discuss if you see any issues. Almost all images were sourced from https://cloud.orga.emfcamp.org, one comes from a village's photo album.

Other notes:

  • All images are optimised for web, i.e. compressed jpegs and resolution reduced to 1080p.
    • Where images are displayed in reduced size I generated reduced resolution thumbnails which drops file size ~80%, but the user can click to view the 1080p version.
  • I still plan to update some minor details of the text, namely I want to see if I can find some more recent press/blog coverage to replace the old links with. I'll take the PR out of draft once I've done that.

@Mattsi-Jansky Mattsi-Jansky marked this pull request as draft April 9, 2026 14:12
@Mattsi-Jansky Mattsi-Jansky marked this pull request as ready for review April 10, 2026 12:32
@Mattsi-Jansky Mattsi-Jansky changed the title [Draft] Update images for about page, refresh the layout Update images for about page, refresh the layout Apr 10, 2026
@russss
Copy link
Copy Markdown
Member

russss commented Apr 12, 2026

I don't like the body text styling here - it should be consistent with the rest of the website even if you think that looks worse. In general any page-specific styling is just going to make it more difficult when it comes to revising the CSS (which we're aiming to do after this year), so I'd like to keep this to an absolute minimum.

I'm inclined to lose the YouTube video at this point, although I'd welcome other opinions.

Copy link
Copy Markdown
Member

@russss russss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(Marking as "request changes" because I always forget to do this and it requires a comment)

@Mattsi-Jansky
Copy link
Copy Markdown
Contributor Author

I don't like the body text styling here - it should be consistent with the rest of the website even if you think that looks worse. In general any page-specific styling is just going to make it more difficult when it comes to revising the CSS (which we're aiming to do after this year), so I'd like to keep this to an absolute minimum.

I'm inclined to lose the YouTube video at this point, although I'd welcome other opinions.

That is a fair point, it'll be simpler to maintain the whole site if we don't make exceptions. I removed the line-height change and custom title/h1 style in eb89b70. Other than those all the text is consistent with the rest of the site, with the exception of the sub-heading.

I do think the sub-heading adds something valuable, but there isn't any similar style across the website to use in its place. I think we can either: a) push it up to _base.scss as a shared subheading style or b) remove the subheading altogether. I think it adds value but I'm happy to remove it if you prefer. What do you think?

Re: the YouTube video personally I think it adds quite a bit, though it could really do with updating as it'll be a decade old come EMF 2028. Happy to remove it as part of this PR if that's what people prefer though.

@jellybob
Copy link
Copy Markdown
Contributor

Chipping in here with some purely subjective feedback. I would personally drop the smaller images sitting to the right of the text and stick with just the large ones. Maybe stick a little grid of smaller ones down the bottom for the people who want to see more. The ones on the right currently for me throw off the page flow a bit.

@Mattsi-Jansky
Copy link
Copy Markdown
Contributor Author

Chipping in here with some purely subjective feedback. I would personally drop the smaller images sitting to the right of the text and stick with just the large ones. Maybe stick a little grid of smaller ones down the bottom for the people who want to see more. The ones on the right currently for me throw off the page flow a bit.

The main impetus for updating this page has been the difficulty in explaining what EMF is to potential sponsors, and enticing their interest. The previous page had a lot of images too, but they were quite old and didn't capture the scale that recent EMFs have reached. I don't think the page would be very enticing without the images, giving visitors a more tangible sense of what the event is like.

@futureshape
Copy link
Copy Markdown
Contributor

I also vote for more images, it's hard to convey the scale of what happens at EMF with a limited set of imagery.

We have a fairly narrow content area and no components (yet) to do carousels/image galleries so whilst we can still improve this in the future, it's a good step forward.

P.S. It would be good to know from the beginning that point was to explain the event to sponsors, I was wondering why we're trying to improve the "about" page for an already sold-out festival :)

@russss
Copy link
Copy Markdown
Member

russss commented May 3, 2026

There's still a line-height: 1.7 on some of the body text which I'd prefer to get rid of. Please can you also squash this PR into one commit, as we don't have the ability to do this with the merge queue, then I'll get it merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants