Skip to content
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

Auto hide dropdown menu #32

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

Conversation

whalekeykeeper
Copy link
Collaborator

@whalekeykeeper whalekeykeeper commented Nov 1, 2024

There are a few changes:

  1. added JS script for auto-hiding the dropdown menu
  2. added CSS code accordingly
  3. update the url setting in _includes/header.html to avoid using local host url, for example, href="/contact/" instead of href="http://localhost:4000/contact/"
  4. update gitignore for preventing committing _site
  5. a small fix for team photo url in team.md

Checked in the following ways:

  • run bundle exec jekyll serve on terminal and examine the given server address to see if the dropdown menu could hide automatically.
  • run bundle exec jekyll serve --host 0.0.0.0 to bind the server to local machine’s IP address and make the website accessible on local network, then use something like http://192.168.1.10:4000/ (replace 192.168.1.10 with the correct computer's IP address) on mobile device to double check if the dropdown menu could hide automatically.
  • run private/incognito browsing mode on both PC and mobile device to check the dropdown menu's hiding feature.
  • check the spread menu in PC browser when it is not in the collapsed status.

see #31 #15

whalekeykeeper and others added 12 commits November 1, 2024 23:55
Fixed a misplaced closing parenthesis.
1. add the script for hiding dropdown menu
2. use relative_url instead of absolute_url
3. add CSS code for hiding dropdown menu
update gitignore
update url
Fixed a misplaced closing parenthesis.
1. add the script for hiding dropdown menu
2. use relative_url instead of absolute_url
3. add CSS code for hiding dropdown menu

see #15
update gitignore
update url
@whalekeykeeper
Copy link
Collaborator Author

whalekeykeeper commented Nov 2, 2024

The dropdown menu hiding feature is successfully added. Though a issue were just found in this branch: when clicking on "Team" or "About" and so on with a PC browser, with the local server, the entire menu part is gone, which is not the case for the current deployed version. I have a guess. Gonna have a look in the next a few days.

@damianodamiani
Copy link
Collaborator

hi Qin! thanks for looking into that. I am not sure what Lena told you, but basically the website as it is now works great on computer screens -- it does not, however, on mobile. I essentially wrote the whole page from zero with some GPT help, so for sure there some weird solutions, and that's also why the mobile version didn't really work. I tried implementing some exisitng frameworks via gh pages or other prewritten packages, but none works with the page. hope you will have some ideas!

My approach would be to either a) make the mobile version be the same as the regular, just with the menu as a dorpdwon etc, or b) make the website recognize its on mobile and load a separate verison, that is much simplified and works specifically on mobile.

What do you think? + also any suggestions for the website are welcomed! I am working now on a TEAM page, could also brainstorm that together if you want :)

@whalekeykeeper
Copy link
Collaborator Author

hi Qin! thanks for looking into that. I am not sure what Lena told you, but basically the website as it is now works great on computer screens -- it does not, however, on mobile. I essentially wrote the whole page from zero with some GPT help, so for sure there some weird solutions, and that's also why the mobile version didn't really work. I tried implementing some exisitng frameworks via gh pages or other prewritten packages, but none works with the page. hope you will have some ideas!

My approach would be to either a) make the mobile version be the same as the regular, just with the menu as a dorpdwon etc, or b) make the website recognize its on mobile and load a separate verison, that is much simplified and works specifically on mobile.

What do you think? + also any suggestions for the website are welcomed! I am working now on a TEAM page, could also brainstorm that together if you want :)

I totally agree with you that to take mobile devices or smaller screens into consideration is actually quite necessary. Many people are using tablets and mobile phones. And it is amazing how much you have already designed and built up!

From my view, a) will be simpler. And that's what I was trying to do. I will have another look later today and come back here.

For TEAM page, if you have any idea, just bring it up, I am not particularly an expert on frontend, but 众人拾柴火焰高 or 人多力量大 (actually the first one jumped into my head is 三个臭皮匠,顶过诸葛亮 haha), therefore, if you have any ideas/questions, feel free to open issues/write me/set up a meeting or whatever.

@damianodamiani
Copy link
Collaborator

OK! lets focus first on the mobile page rendering, the team page can come after :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants