Anime-Girls-Holding-Programming-Books icon indicating copy to clipboard operation
Anime-Girls-Holding-Programming-Books copied to clipboard

Why don't you make a Github Pages website?

Open Kolahzary opened this issue 5 years ago • 25 comments

You do well, but I think it would be better if combined with a simple html

Kolahzary avatar Apr 12 '19 15:04 Kolahzary

It'd be good to make this page access through github api's to get all files and not copypasting

ske2004 avatar May 03 '19 19:05 ske2004

Off-topic, but are you the Shadman?

Asday avatar May 03 '19 19:05 Asday

Do you mean like some kind web page where you could view the images more easily.

If thats what you mean I would definitely be open to doing that, and adding it as project for the repo.

cat-milk avatar Jun 07 '19 18:06 cat-milk

I've been thinking of doing something along those lines: having a sidebar with the name of the girls and a "selection screen" beside it that displays the character, similar to how the character selection pans out in fighting games. I think it wouldn't be too difficult to make something presentable.

If going on that direction, I can think of a few good libraries that enable this: https://github.com/element-motion/element-motion https://www.react-spring.io/

And aside from that, a carousel that fills the remaining of the screen. That's it.

resolritter avatar Jun 10 '19 02:06 resolritter

As a professional web developer, heck c*rousels.

Asday avatar Jun 10 '19 02:06 Asday

True! It could be a long container with vertical scrolling; that would make it easier to download everything at once... Not that carousels make it difficult. It's just the first thing that popped out on me because I'm thinking of fighting games' costume selection.

I'll see if I can get something going tomorrow.

resolritter avatar Jun 10 '19 02:06 resolritter

For the record I did try to start, but I'm so out of touch with front-end development that I ended up spiralling into depressive self-unworthiness (this actually happened). I can't go as fast as I could, thinking of modelling components and such, so it's a hurdle. If I do get back on my feet at front-end, I'll try once again.

resolritter avatar Jun 28 '19 19:06 resolritter

Sounds like you're overthinking it. Pick a Hugo gallery template, tweak it a bit, and you're done.

Asday avatar Jun 28 '19 20:06 Asday

I'm interested in working on this so I made a rough mockup. The colors and overall layout will probably change but this is as far as I can push my creative boundaries at 9am.

ghpb

There are certain images that are extremely off center so we might need to keep some configuration files for problematic images to center them manually. It's not a huge deal though, as we might already want to keep metadata of images like what character is in the picture or what anime it's from to avoid this meme

Let me know what you think, happy to do tweaks or start working on the real thing. It would most likely be in Gatsby to address problems like gigantic image sizes and loading of the metadata I mentioned earlier. We could even go as far as to offer zip files of the entire archive if that's something you would be interested in.

EDIT: Also I noticed that you guys are starting to run out of space so we could eventually override the images with the .webm files Gatsby generates to cut the repo size down drastically and add more girls (or boys).

Xetera avatar Jul 07 '19 17:07 Xetera

that looks really good. do you think we should start a separate repo for the pages source or would cramming it into a folder on this repo be better

cat-milk avatar Jul 07 '19 17:07 cat-milk

It's probably better to keep everything in one place because this repo will be the "one source of truth" so just making a web folder or something would probably work better for this. I can get a basic preview going on netlify for my own fork and if you're happy with the overall layout you can make a separate branch and I'll PR it :+1:

Xetera avatar Jul 07 '19 17:07 Xetera

We can always stuff it into the gh-pages branch if it can be deployed directly or another branch if it can't, but I agree with keeping it in this repo. I wouldn't want it in the master branch, though.

bucket3432 avatar Jul 07 '19 22:07 bucket3432

Also I noticed that you guys are starting to run out of space

@Xetera where do you see that we're running out of space? The repo is only at 88MB, which is way under the recommended 1GB limit (and the hard limit is 100GB, which is even further off). If you're thinking that the limit is 100MB, that's for individual files.

bucket3432 avatar Jul 07 '19 22:07 bucket3432

That's not what branches are for though...

I don't see why you wouldn't want master to contain the website. It would stay up to date with new pictures much more easily that way, and all it takes is one index.html at the root or in docs/.

Asday avatar Jul 07 '19 22:07 Asday

That's not what branches are commonly used for, but it's still a valid use of a branch. It's perfectly valid to say that all Web-related code is developed on another branch. One might argue that that's a reason for it to be in another repo, and that's also a perfectly fine option as well. The only argument I have for it being in the same repo is that it's tightly coupled with this repo; if the site is general enough that it's only loosely coupled, then it should go in a different repo.

I'd rather keep it out of master because that's not really the deliverable of this repo, unless we say that the site can be run locally and is the preferred way to view the images in the repo.

bucket3432 avatar Jul 07 '19 22:07 bucket3432

What's your definition of "valid use"? Sure it would work, but it would be a lot more work to maintain. Why make that harder?

I'm speaking from the experience of dealing with redux-form, where the website is stored on a different branch. It's certainly not how it should be done if you expect people to be using github in relation to your repo, which, being on github, we do.

Asday avatar Jul 07 '19 22:07 Asday

@bucket3432 oh whoops, I guess I misread the earlier conversation. In that case, it doesn't really matter if the images themselves are large or not.

I was thinking it would be more convenient to deploy the website on Netlify. There wouldn't need to be any html file involved in the repo at all, just the source code for the website. The repo I have now with a semi-working website is just the exact setup you guys have in this repo except with a website folder in the root. I don't think that would be problematic, would it? I don't think there's any need to loosely couple anything since the website is specifically only for this repo.

Xetera avatar Jul 07 '19 22:07 Xetera

If you literally rename website/ to docs/, then it'll work fine on github pages. No need to add on some third party service.

Asday avatar Jul 07 '19 22:07 Asday

That works too, but then we'll need travis or circle CI to compile the react code into html and push to a new branch. It's also possible to do it with pure html and js and not bother with a static site generator but then we'd be serving 12M of images for C and 31M for SICP, I don't think anyone would like that. I'm fine with setting up a simple build step with Travis CI, it already supports this kind of thing by default but there's a tiny bit of configuration to be done on the repo owner's end.

Xetera avatar Jul 07 '19 22:07 Xetera

I didn't think about thumbnailing, you are right. Shame there's no good service for that.

Asday avatar Jul 07 '19 22:07 Asday

if you expect people to be using github

Alright, I concede the point in the context of GitHub. In that case, I don't have any strong reasons for or against either way.

As for thumbnailing, we can probably run something on the CI service to resize the image, or find a good CDN that will do it for us.

bucket3432 avatar Jul 07 '19 23:07 bucket3432

"Good, free, pick one."

I could run a service on my home server, but then my internet goes out every so often, and I'm moving house soon.

I don't imagine anyone here's looking to pay, either.

Asday avatar Jul 07 '19 23:07 Asday

Netlify lets you do all of this in about 4 clicks, it's also free. The resizing is taken care of by gatsby so you don't have to worry about a cdn or any other CI tool for that. I've just tested it on my end and the 12Mb C folder is shrunk all the way down to 400kb with full quality 250x250 webp previews :+1:

As for netlify, you just sign in with github, click on the repo and it will detect the netlify configuration in the repo and figure out how to build and deploy the website on its own, it's really convenient.

The tradeoff here (since there's always one) is long compile times. Gatsby tries to generate around 800 different sized images along with a separate page for each folder while compiling so a proper build takes a solid 5-7 minutes on a slow netlify machine.

Xetera avatar Jul 07 '19 23:07 Xetera

This repo doesn't get updates too often, so I don't expect long build times to be an issue. I'd be more worried about how someone can test their changes locally.

bucket3432 avatar Jul 07 '19 23:07 bucket3432

Netlify supports hugo too, right? Hugo's build times are measured in milliseconds, from my (admittedly limited) experience.

Asday avatar Jul 07 '19 23:07 Asday

I'll have instructions for all that in the website folder. A single npm run develop should be enough to get the website up locally if you ever want to test it. Netlify also creates previews of the new website when pull requests are created so before new images are added you should be able to see what the new website will look like.

From what I know, when you introduce tooling to Hugo for processing css and minifying assets and such, the build times are comparable to Gatsby's. I haven't looked into Hugo much, I just prefer Gatsby because React is fantastic :ok_hand:

Xetera avatar Jul 07 '19 23:07 Xetera

I have a basic preview up on https://anime-girls-holding-programming-books.netlify.com it's not styled yet but it's got the basic functionality for now

Xetera avatar Jul 08 '19 02:07 Xetera

WOWOWOWOW!!! EXCITED!!! I am going to make one too!!!!! On it!!!

shrekuu avatar May 22 '22 07:05 shrekuu