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

Make darkfish more responsive and readable on mobile devices #1162

Merged
merged 1 commit into from
Aug 26, 2024

Conversation

st0012
Copy link
Member

@st0012 st0012 commented Aug 24, 2024

  • Make the sidebar toggle fixed on all devices
  • Prevent default zooming on mobile devices
  • Improve sidebar opening on mobile devices

Before

Screen.Recording.2024-08-24.at.18.58.59.mov

After

Screen.Recording.2024-08-24.at.19.00.41.mov

Comment on lines +222 to +223
display: block;
position: fixed;
Copy link
Contributor

Choose a reason for hiding this comment

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

Non-blocking: I actually like the sidebar on the left, even if it moves. I think most websites work like that.

Copy link
Member Author

Choose a reason for hiding this comment

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

Sidebar is still on the left.

Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry. I meant the icon.

Copy link
Contributor

Choose a reason for hiding this comment

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

In any case, we could add a background color to the icon, so when it is on top of text it is still visible. I think I had that in the first PR

Copy link
Member Author

Choose a reason for hiding this comment

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

I think the right solution is to properly implement a sticky navbar at the top when on small screens that displays the button on the left as you suggest, and the nav icon will never hover over text.
Before that happens, I think putting the button to the right on mobile is less intrusive.

@st0012 st0012 force-pushed the improve-responsive branch from 9dbe09d to cacf83c Compare August 25, 2024 11:07
@st0012 st0012 requested a review from MatheusRich August 25, 2024 11:10
@MatheusRich
Copy link
Contributor

@st0012 This PR definitely helps! TY.

On a different note, do you have thoughts on making the colors more similar to Ruby API's? I'm happy to work on that.

@st0012 st0012 force-pushed the improve-responsive branch from cacf83c to 692efe3 Compare August 25, 2024 22:32
@st0012
Copy link
Member Author

st0012 commented Aug 25, 2024

On a different note, do you have thoughts on making the colors more similar to Ruby API's?

I think it'd be beneficial to explore multiple color options, which I also tried a few myself recently (and failed to make anything else work).

I will appreciate your attempt for sure and @colby-swandale as the author of rubyapi.org will provide more constructive opinion than me 🙂

- Make the sidebar toggle fixed on all devices
- Prevent default zooming on mobile devices
- Improve sidebar opening on mobile devices
@st0012 st0012 force-pushed the improve-responsive branch from 692efe3 to 750fdfc Compare August 26, 2024 10:32
@st0012 st0012 requested a review from colby-swandale August 26, 2024 10:33
@st0012 st0012 merged commit 95b6cfb into master Aug 26, 2024
51 checks passed
@st0012 st0012 deleted the improve-responsive branch August 26, 2024 22:02
matzbot pushed a commit to ruby/ruby that referenced this pull request Aug 26, 2024
devices
(ruby/rdoc#1162)

- Make the sidebar toggle fixed on all devices
- Prevent default zooming on mobile devices
- Improve sidebar opening on mobile devices

ruby/rdoc@95b6cfb64f
@st0012 st0012 added this to the v6.8.0 milestone Oct 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants