Margins are too narrow on mobile #655

Closed
opened 5 years ago by marek-lach · 2 comments
marek-lach commented 5 years ago (Migrated from github.com)

This is not a high priority issue, but compared with the old theme, the left/right margins of a blog post are too narrowly close to the edges of the screen on mobile, especially on the 21:9 aspect ratio screens etc...

You can see the old margins pre ##624:

IMG_0101

And after:

IMG_0100

I have not seen any actual text being cropped just yet, but it'd be a good idea to have some small margin distance from the display's edges.

  • Plume version: latest commit
  • Related issues: #650
  • Operating system: iOS 12/iPhone X, but I've also tested it on Android 9
  • Web Browser: iOS Safari, and also Firefox and Chrome for Android
<!-- Describe your bug, explaining how to reproduce it, and what was expected --> This is not a high priority issue, but compared with the old theme, the left/right margins of a blog post are too narrowly close to the edges of the screen on mobile, especially on the 21:9 aspect ratio screens etc... You can see the old margins pre ##624: ![IMG_0101](https://user-images.githubusercontent.com/45913977/63687588-46385780-c805-11e9-8d8a-d79081df40e7.PNG) And after: ![IMG_0100](https://user-images.githubusercontent.com/45913977/63687630-5cdeae80-c805-11e9-9880-cae25ee39039.PNG) I have not seen any actual text being cropped just yet, but it'd be a good idea to have some small `margin` distance from the display's edges. - **Plume version:** latest commit - **Related issues:** #650 - **Operating system:** iOS 12/iPhone X, but I've also tested it on Android 9 - **Web Browser:** iOS Safari, and also Firefox and Chrome for Android
marek-lach commented 5 years ago (Migrated from github.com)

Closed by #662 (and better by #663).

Closed by #662 (and better by #663).
elegaanz commented 5 years ago (Migrated from github.com)

Sorry, but I reopen for the moment, as long as the PR has not been merged.

Sorry, but I reopen for the moment, as long as the PR has not been merged.
KitaitiMakoto closed this issue 3 years ago
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Plume/Plume#655
Loading…
There is no content yet.