NatLabRockies / NatLabRockies/SAM

LK Script progress bar does not resize to fit text

Open
#2,122 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
C++
Stars
486
Forks
219
Avg merge
10h 10m
Merged PRs (30d)
2

Description

Describe the bug

The LK Script progress bar's Close button is hidden when text has more than two lines. For example, in Windows:

Image
To Reproduce
  1. Download and extract this LK script: lk-progressbar-size-test.zip
  2. Run the script and note dialog does not resize to fit text.
Expected behavior

Best solution would be to add size parameters to function so that the progressbar size can be set when the function is called. We want the progress dialog size to be static so it does not appear to move around the screen when the text changes.

Needs to work for Windows, Linux, and Macos, so pixel-based sizing is not ideal due to different font sizes across the platforms.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by running the linked lk-progressbar-size-test.zip reproduction and inspect the LK Script progress-bar implementation. Determine how dialog sizing is chosen on Windows, Linux, and macOS, then verify that configurable static sizing keeps the text and Close button visible without moving when the text changes.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
desktop
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.