Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Aug 16, 2025

The column layout sidebar (containing journal queries) was not visible on common display setups with browser scaling, forcing users to zoom out which made text hard to read.

Problem

  • Plugin required minimum viewport width of 1850px to show the sidebar
  • Users with 1920x1080 displays using 1.25x browser scaling have ~1536px effective viewport width
  • Sidebar remained hidden in default mode, only appearing when zoomed out (making text unreadable)

Solution

Lowered the minimum width requirement from 1850px to 1400px and added a new responsive breakpoint:

Before:

@media screen and (min-width:1850px) {
  /* Column layout only activated at 1850px+ */
}

After:

@media screen and (min-width:1400px) {
  /* Column layout now activates at 1400px+ */
}

@media screen and (min-width:1400px) and (max-width:1849px) {
  /* New breakpoint for smaller screens */
  margin-right: 300px;
  
  & #today-queries {
    width: 320px;
  }
  
  & div.journal>div.flex.flex-col {
    min-width: 700px
  }
}

Changes Made

  • Updated src/css/side.css to use 1400px minimum width and added new breakpoint
  • Updated src/css/notSide.css for consistency when linked references are disabled
  • Updated documentation in README.md to reflect new requirement

Testing

For a 1920x1080 display with 1.25x scaling:

  • Effective viewport width: 1920 ÷ 1.25 = 1536px
  • Layout space usage: 700px (journal) + 320px (sidebar) + 300px (margin) = 1320px
  • Available space: 1536px ✅ (216px remaining for comfortable layout)

This makes the plugin usable for users with common scaled display configurations without requiring zoom adjustments.

Fixes #28.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copy link
Contributor Author

Copilot AI commented Aug 16, 2025

@YU000jp 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

Copilot AI and others added 3 commits August 16, 2025 06:56
Co-authored-by: YU000jp <111847207+YU000jp@users.noreply.github.com>
…ibility

Co-authored-by: YU000jp <111847207+YU000jp@users.noreply.github.com>
…um width

Co-authored-by: YU000jp <111847207+YU000jp@users.noreply.github.com>
@Copilot Copilot AI changed the title [WIP] Sidebar not shown unitl zoomed out, then text becomes very hard to read Lower minimum width requirement from 1850px to 1400px for sidebar visibility on scaled displays Aug 16, 2025
@Copilot Copilot AI requested a review from YU000jp August 16, 2025 07:03
Copilot finished work on behalf of YU000jp August 16, 2025 07:03
@YU000jp YU000jp marked this pull request as ready for review August 16, 2025 07:05
@YU000jp YU000jp merged commit f99e841 into main Aug 17, 2025
github-actions bot pushed a commit that referenced this pull request Aug 17, 2025
## [2.13.1](v2.13.0...v2.13.1) (2025-08-17)

### Bug Fixes

* Lower minimum width requirement from 1850px to 1400px for sidebar visibility on scaled displays [#28](#28) ([f99e841](f99e841)), closes [#29](#29)
Copy link
Contributor

🎉 This PR is included in version 2.13.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

Sidebar not shown unitl zoomed out, then text becomes very hard to read
2 participants