Skip to content

Fix: show room symbols at smaller sizes#7593

Merged
vadi2 merged 2 commits intoMudlet:developmentfrom
SlySven:Fix_showRoomSymbolsAtSmallerSizes
Dec 27, 2024
Merged

Fix: show room symbols at smaller sizes#7593
vadi2 merged 2 commits intoMudlet:developmentfrom
SlySven:Fix_showRoomSymbolsAtSmallerSizes

Conversation

@SlySven
Copy link
Copy Markdown
Member

@SlySven SlySven commented Dec 25, 2024

Brief overview of PR changes/additions

Allow the font used to draw room symbols to be half the previous minimum so that they show at even smaller sizes and thus are shown when the map is zoomed out more.

Motivation for adding to Mudlet

Someone (@Kebap) was unhappy at the size at which symbols were not being shown.

Other info (issues closed, discussion etc)

If accepted this will close #5320 and allow me to /claim #5320.

#### Brief overview of PR changes/additions
Allow the font used to draw room symbols to be half the previous minimum so
that they show at even smaller sizes and thus are shown when the map is
zoomed out more.

#### Motivation for adding to Mudlet
Someone (@Kebap) was unhappy at the size at which symbols were not being
shown.

#### Other info (issues closed, discussion etc)
If accepted this will close Mudlet#5320 and allow me to /claim Mudlet#5320.

Signed-off-by: Stephen Lyons <slysven@virginmedia.com>
@SlySven SlySven requested a review from a team as a code owner December 25, 2024 14:48
@add-deployment-links
Copy link
Copy Markdown

add-deployment-links bot commented Dec 25, 2024

Hey there! Thanks for helping Mudlet improve. 🌟

Test versions

You can directly test the changes here:

No need to install anything - just unzip and run.
Let us know if it works well, and if it doesn't, please give details.

@Kebap
Copy link
Copy Markdown
Contributor

Kebap commented Dec 26, 2024

For me, this works fine. Compare this to the behavior shown in the original issue:

Test5320

Not only do the symbols hide later, they also hide at the same time, and don't arbitrarily reappear neither.

This change will likely influence other folks who write stuff on their rooms, so needs more testing and review in PTB.

@Kebap Kebap removed their assignment Dec 26, 2024
@SlySven
Copy link
Copy Markdown
Member Author

SlySven commented Dec 26, 2024

Not only do the symbols hide later, they also hide at the same time, and don't arbitrarily reappear neither.

That means the other limit is being reached - the one for the overall size of rooms (8 pixels IIRC) rather than the font limit - the latter will still come into play for a symbol that is more than one or two graphemes I expect...

@vadi2
Copy link
Copy Markdown
Member

vadi2 commented Dec 26, 2024

I thought there was more to the issue at hand. Nice job solving this simply 👍

@vadi2 vadi2 merged commit 0ba66c9 into Mudlet:development Dec 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Zooming unexpectedly hides room symbols

3 participants