Skip to content

Statusbar: ensure that statusbar elements can be controlled via preferences - make field properly sized for its contents#3608

Merged
Jarod42 merged 1 commit intoeranif:masterfrom
UffeJakobsen:master_uj_statusbar_field_size
Mar 20, 2025
Merged

Statusbar: ensure that statusbar elements can be controlled via preferences - make field properly sized for its contents#3608
Jarod42 merged 1 commit intoeranif:masterfrom
UffeJakobsen:master_uj_statusbar_field_size

Conversation

@UffeJakobsen
Copy link
Copy Markdown
Contributor

It gets rather lengthy...

-    int lineColWidth = GetTextWidth("Ln 100000, Col 999, Pos 12345678, Len 4821182");
+    int lineColWidth = GetTextWidth("Ln 123456, Col 123, Pos 12345678, Len 12345678, Sel 1234567, SelLn 12345");

Another approach could be to not display "Pos" and "Len" - if a selection is ongoing ?

@UffeJakobsen UffeJakobsen force-pushed the master_uj_statusbar_field_size branch from 1f6e06d to fdf5ada Compare March 19, 2025 16:10
Copy link
Copy Markdown
Owner

@eranif eranif left a comment

Choose a reason for hiding this comment

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

What to show is controllable from settings -> preferences -> misc -> status bar
You can make new entry for the new addition and let the user decide what he wants to display

…rences - make field properly sized for its contents
@UffeJakobsen UffeJakobsen force-pushed the master_uj_statusbar_field_size branch from fdf5ada to ffb08c9 Compare March 20, 2025 15:07
@UffeJakobsen UffeJakobsen changed the title Statusbar: ensure that statusbar field is properly sized for its contents Statusbar: ensure that statusbar elements can be controlled via preferences - make field properly sized for its contents Mar 20, 2025
@Jarod42 Jarod42 merged commit 8586f3b into eranif:master Mar 20, 2025
4 checks passed
@UffeJakobsen UffeJakobsen deleted the master_uj_statusbar_field_size branch March 21, 2025 17:05
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.

3 participants