Skip to content

Improve UI compactness and hide persistent status indicator#15

Merged
asim merged 2 commits intomasterfrom
copilot/fix-point-listing-visibility
Dec 28, 2025
Merged

Improve UI compactness and hide persistent status indicator#15
asim merged 2 commits intomasterfrom
copilot/fix-point-listing-visibility

Conversation

Copy link
Contributor

Copilot AI commented Dec 28, 2025

The UI had three issues: form controls occupied excessive vertical space preventing scrolling to view all points, the status box permanently displayed "Ready" cluttering the interface, and input fields were unnecessarily large.

Changes

Form compactness

  • Reduced input widths: X/Y from 80px → 60px, Label from 120px → 100px
  • Reduced padding and min-height: 0.6rem/44px → 0.4rem 0.5rem/36px
  • Tightened header spacing: 1rem → 0.75rem, reduced h1 size
  • Reduced control group gaps: 0.5rem → 0.3rem

Status indicator behavior

Changed from persistent "Ready" text to auto-hiding notifications:

.status {
    display: none;  /* Hidden by default */
}
.status.visible {
    display: block;  /* Only shown during messages */
}
function setStatus(message, duration = 3000) {
    status.classList.add('visible');
    setTimeout(() => status.classList.remove('visible'), duration);
}

Status now appears only when showing messages (e.g., "✓ Loaded 18 points") and auto-hides after 3 seconds.

Screenshots

Before - Compact form, no persistent status:
Compact form

During action - Status briefly appears:
Status visible

After - Status auto-hidden, scrollable sidebar:
Final state

Original prompt

i cant really scroll thr page to see thr point listing. And the ready box doesn't go away. Maybe the x,y point and label boxes could be smaller or the whole form and buttons could be smaller.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

- Reduced input field widths (X/Y: 80px -> 60px, Label: 120px -> 100px)
- Reduced button and input padding (0.6rem -> 0.4rem 0.5rem)
- Reduced min-height for inputs/buttons (44px -> 36px)
- Reduced header padding (1rem -> 0.75rem)
- Reduced h1 font size (1.5rem -> 1.25rem)
- Reduced gap in control groups (0.5rem -> 0.3rem)
- Made status box hidden by default, only shows when messages appear
- Status box automatically hides after 3 seconds instead of showing "Ready"
- Sidebar already has overflow-y: auto for proper scrolling

Co-authored-by: asim <17530+asim@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix page scrolling issue with point listing Improve UI compactness and hide persistent status indicator Dec 28, 2025
Copilot AI requested a review from asim December 28, 2025 23:55
@asim asim marked this pull request as ready for review December 28, 2025 23:55
@asim asim merged commit 35ad4a4 into master Dec 28, 2025
@asim asim deleted the copilot/fix-point-listing-visibility branch December 28, 2025 23:55
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