Skip to content

Conversation

@siliconsniffer
Copy link
Contributor

@siliconsniffer siliconsniffer commented Dec 22, 2025

Add compact, short information about your PR for easier understanding:

Note

I used an LLM Chatbot/Assistant during development, but I take full responsibility for the submitted changes.

This is a follow-up PR of #16749.

  • Goal of the PR
    Display status messages in the main menu.
  • How does the PR work?
    Implements a new GUI component GUIStatusText for displaying temporary status messages.
  • Does it resolve any reported issue?
    Closes [Formspec] Status messages in the main menu (e.g. for screenshots) #16759
  • Does this relate to a goal in the roadmap?
    Not directly, as far as I can tell.
  • If not a bug fix, why is this PR needed? What usecases does it solve?
    It hints to the user that a screenshot has been made.

To do

This PR is ready for review.

  • Styling of the status message?
  • Test on Android.

How to test

open luanti → press F12 / screenshot keybind → observe that a status message appears and fades out again

Preview

screenshot_20251222_213642

@sfan5
Copy link
Collaborator

sfan5 commented Dec 22, 2025

Styling of the status message?

I think the black background is unnecessary.

@sfan5 sfan5 added Feature ✨ PRs that add or enhance a feature @ Mainmenu / Settingsmenu labels Dec 22, 2025
@CrazyladMT
Copy link
Contributor

CrazyladMT commented Dec 22, 2025

I think the black background is unnecessary.

I think it looks fine. Without it, the text would be unreadable on the eye cancer shade of blue used for the menu background.

@siliconsniffer
Copy link
Contributor Author

siliconsniffer commented Dec 22, 2025

I think the black background is unnecessary.

I disagree. On a smaller screen, the text will overlap with the gamebar and will be hardly readable.
screenshot_20251222_231926
For comparison, here it is with the background.
screenshot_20251222_232217

@siliconsniffer
Copy link
Contributor Author

It does work on Android, but while testing I noticed two things:

  • the long path renders the status message practically worthless
  • in contrast to the opaque background shown above, it is transparent

@Zughy
Copy link
Contributor

Zughy commented Dec 23, 2025

Wait, why do you need to notice players on Android? Doesn't the OS already play an animation on its own when taking a screenshot?

@siliconsniffer siliconsniffer force-pushed the status_message branch 2 times, most recently from 15587b6 to 2a5061e Compare December 26, 2025 22:53
@siliconsniffer siliconsniffer marked this pull request as draft December 27, 2025 10:32
@sfan5
Copy link
Collaborator

sfan5 commented Dec 27, 2025

Wait, why do you need to notice players on Android? Doesn't the OS already play an animation on its own when taking a screenshot?

This is about the F12 screenshot function in Luanti. This is not even accessible on Android unless you have a keyboard connected.

@siliconsniffer

This comment was marked as outdated.

@siliconsniffer siliconsniffer changed the title Add status messages for screenshots in the main menu Show status text for screenshots in the main menu Dec 27, 2025
@siliconsniffer
Copy link
Contributor Author

A slightly simpler design:
screenshot_20251227_143233_1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature ✨ PRs that add or enhance a feature @ Mainmenu / Settingsmenu

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Formspec] Status messages in the main menu (e.g. for screenshots)

4 participants