Skip to content

Properly unify nav and mobile nav structure, fix markup issues #98

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

Merged
merged 1 commit into from
Jul 30, 2021

Conversation

jhrcek
Copy link
Contributor

@jhrcek jhrcek commented Jul 30, 2021

Hello @myShoggoth
this is a followup to your change #87 which, with all due respect, was not implemented correctly.
You seem to have modified the html in docs directory directly.
But the proper way was to fix the templates under site and then generate the contents of the docs folder by running ./build.sh in the root of the repo.

I did just that, respecting your changes and including few general markup fixes as follows

  • the divs in the mobile nav had href attribute - this is not valid html and correspondingly had no effect (in the sense that those divs didn't work as a link)
  • the mobile nav links were missing some css classes, so they were not properly nested - I added those.
  • there were couple of elements with empty id attribute - I removed those
  • the haskell foundation logos in the footer had a dead link - this has now been fixed by running ./build.sh

Note: to understand what I changed just look at the files under site/templates. The rest of the changes (everything under docs) are just result of regenerating the site by running hakyll .

@tomjaguarpaw
Copy link
Contributor

@jhrcek Are you saying that the docs/ directory is generated from site/ by running ./build.sh? If so, does anyone know why docs/ is checked in at all? (Perhaps that's how Github pages works?)

@elland
Copy link
Contributor

elland commented Jul 30, 2021

@tomjaguarpaw

(Perhaps that's how Github pages works?)

That's my understanding as well.

@jhrcek thanks for catching that.

@jhrcek
Copy link
Contributor Author

jhrcek commented Jul 30, 2021

Yes. I know that github pages allows you to publish html from selected subfolder on master branch of your repository.

Is that how this particular repository is setup? I can't say, because I don't have access to its settings, but I think it's done that way. Can someone else who has those access rights please confirm that?

Here's how these settings look in my personal github pages -enabled repo:
Screenshot from 2021-07-30 15-15-53

@tomjaguarpaw
Copy link
Contributor

@myShoggoth
Copy link
Contributor

Thank you for fixing my mess!

@myShoggoth myShoggoth merged commit 0a87aee into haskellfoundation:master Jul 30, 2021
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