Giter Club home page Giter Club logo

Comments (19)

ngdangtu-vn avatar ngdangtu-vn commented on August 11, 2024 1

Oh, too bad for her. Well what done is done. I will attempt to give you a simple menu on mobile design. And after you approve it, I'll start to fix the website.

from website.

TheLastProject avatar TheLastProject commented on August 11, 2024 1

Sounds great then 👍

from website.

obfusk avatar obfusk commented on August 11, 2024 1

This is pretty much what my own website does on mobile, so yeah LGTM.

from website.

TheLastProject avatar TheLastProject commented on August 11, 2024 1

I don't really have a real JS policy because I'm not a web dev. But I'm thinking we can do a first version with the full JS file (because reviewing minified files for security is very hard so for third party contributions I'd rather avoid accepting minified JS) and then later figure out how to best automate JavaScript minification in Jekyll as a separate issue (then we can get rid of the .min.js for bwip too).

Sure, full JS is slightly slower than minified JS but the website is kept pretty minimal so I don't think most users will really notice any real difference so I find it more of a "nice to have" than "must have" to have it minified.

About testing locally, I'll be honest, I kinda gave up on that myself. Jekyll can be hellish to get running. Perhaps people more experienced with Jekyll can help: https://jekyllrb.com/docs/community/#where-to-get-support.

If that doesn't work, feel free to make a draft PR and then ping me in this issue when it's ready for review so I can ignore the emails but you can still make use of the temporary URL Netlify generates.

from website.

TheLastProject avatar TheLastProject commented on August 11, 2024

I thought this was fixed in #18 already but I guess it can still be improved further. If you think you can improve it, go ahead :)

from website.

MaguiRocha avatar MaguiRocha commented on August 11, 2024

I will do my best to improve them! Thank you very much!! ✨

from website.

MaguiRocha avatar MaguiRocha commented on August 11, 2024

Hi !
I need to ask you something, sorry if it's not the place, but it's my first hacktoberfest,
I can't find the html and style code of the error I sent you, I searched inside the repository, but I couldn't find it

from website.

TheLastProject avatar TheLastProject commented on August 11, 2024

The website is built using Jekyll.

The main page is _layouts/default.html. It includes several files from _includes. Specifically, I think you're looking for _includes/header.html.

The CSS is actually SASS and in _sass, this is turned into CSS at build time.

from website.

MaguiRocha avatar MaguiRocha commented on August 11, 2024

Ah Ok 😭
I still don't know Jekyll, I hope to be able to collaborate with your projects next year!

from website.

ngdangtu-vn avatar ngdangtu-vn commented on August 11, 2024

@MaguiRocha You gave me a guity thought of stealing your job for the Fest. You don't need to know Jekyll for this, you only need to understand SCSS and an approved design for this case. About the design, I don't know if they have a design for this case. I can make a design patch for this case if @TheLastProject (which properly not) fire an issue for it and assign to me.

About the CSS, here is my tip for you:

image

Follow the red line, you can see the file you need to modify is in /_sass/layout.scss. Try to mess around a bit more before giving up.

@TheLastProject can I patch your website design?

from website.

TheLastProject avatar TheLastProject commented on August 11, 2024

I don't really have a design, I'm not a designer (I based the website onto the template of someone else), so I don't really have a real strong opinion on what is the best design. My first thought was to allow the header to become multiple lines if it doesn't fit so we can fit more things in it, but maybe a dropdown menu or so is better. If you have a suggested design I'd happily hear about it, it has been a year so sure you can have this assigned :)

from website.

ngdangtu-vn avatar ngdangtu-vn commented on August 11, 2024

Here is purpose patch design:

image

This is common way to display main-menu in mobile. Familiar, and easy to build. I can do it at maximum a half day.

from website.

TheLastProject avatar TheLastProject commented on August 11, 2024

Makes sense, it'll still use the old layout on desktop where there is enough space then?

from website.

ngdangtu-vn avatar ngdangtu-vn commented on August 11, 2024

Of course, no point to hide it in desktop :)

from website.

ngdangtu-vn avatar ngdangtu-vn commented on August 11, 2024

This feature (without CSS hacking) requires JavaScript. And I don't how did you guy deal with JS. So how should I handle the JS? For now I can attempt to do:

  1. HTML/CSS only → this way is ugly to SEO for sure
  2. JS without compressing → I'm doing this but it could harm website speed a little
  3. Compressed JS → increase project complexity which is not making sense for deploy such a tiny feature like this.
  4. Compress JS manually → I plan to write instruction on js file to tell dev upload this file to an online compression server to get min.js (I recommend this case for simple & open website)

What do you think, which do you prefer?

from website.

ngdangtu-vn avatar ngdangtu-vn commented on August 11, 2024

Also, how do you test site in localhost? I tried bundle and got this error:

An error occurred while installing http_parser.rb (0.8.0), and Bundler cannot continue.

In Gemfile:
  jekyll was resolved to 4.3.2, which depends on
    em-websocket was resolved to 0.5.3, which depends on
      http_parser.rb

from website.

obfusk avatar obfusk commented on August 11, 2024

About testing locally, I'll be honest, I kinda gave up on that myself.

FWIW, I just temporarily delete the Gemfile and change :compressed to compressed in _config.yml and then it runs fine locally with the jekyll package from Debian for me 😅

from website.

ngdangtu-vn avatar ngdangtu-vn commented on August 11, 2024

About testing locally, I'll be honest, I kinda gave up on that myself.

FWIW, I just temporarily delete the Gemfile and change :compressed to compressed in _config.yml and then it runs fine locally with the jekyll package from Debian for me 😅

Thanks but "it doesn't work on my machine" :9. Anyway, since the site is not too big,I will pardon you guy netlify for awhile.

from website.

TheLastProject avatar TheLastProject commented on August 11, 2024

Fixed in f221943

from website.

Related Issues (13)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.