Skip to content

use narrow page on narrow devices#344

Merged
paul90 merged 2 commits intomainfrom
ward/narrow
Jul 9, 2025
Merged

use narrow page on narrow devices#344
paul90 merged 2 commits intomainfrom
ward/narrow

Conversation

@WardCunningham
Copy link
Member

@WardCunningham WardCunningham commented Jun 27, 2025

Small adjustments view in the chrome mobile device simulator.

image

This specific complaint is from Thompson who writes,

When I send pages to others with the likelihood they will be read in their phone, I do so being aware that since the last major update, they don’t render well as the text width of the page is larger than the screen size, requiring some annoying side to side scrolling… I wonder if I’m the only one seeing this issue…
http://thompson.forage.ustawi.wiki/view/vagus-nerve

This css is from Baldur Bjarnason who suggested it as a quick fix without getting into page formatting theory.

@WardCunningham WardCunningham requested a review from paul90 June 27, 2025 23:23
@WardCunningham
Copy link
Member Author

image

@WardCunningham
Copy link
Member Author

Robert Best and I explored this today and found that hard coded widths in pixels for some page items prevent the .page from shrinking to the expected width. This seemed to work ok with these widths simply removed. I wonder why they were included if unnecessary?

  • Journal
  • Backlinks

Also for .page we needed to change overflow: auto to be overflow-y: auto.

@WardCunningham
Copy link
Member Author

Commit 60adb2e addresses the width and overflow adjustments discussed yesterday. #344 (comment)

Coding seems a bit safer in this version.

We know that some plugins, such as the Video plugin, don't play by the same rules and will induce additional scrolling in the mobile configuration. Paul, Eric and I have coded these mods together and don't think misbehaving plugins should stop this from going forward.

Copy link
Member

@paul90 paul90 left a comment

Choose a reason for hiding this comment

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

As noted elsewhere, there a few plugins that have the 420px width, but this looks good.

@paul90 paul90 merged commit d78be65 into main Jul 9, 2025
3 checks passed
@paul90 paul90 deleted the ward/narrow branch July 9, 2025 13:39
@paul90
Copy link
Member

paul90 commented Jul 9, 2025

Published as + wiki-client@0.31.3

@WardCunningham WardCunningham restored the ward/narrow branch September 1, 2025 16:17
@WardCunningham WardCunningham deleted the ward/narrow branch September 1, 2025 16:22
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.

2 participants