Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nothing happens when I click [Map > Map Properties...] #3806

Open
manake opened this issue Aug 12, 2023 · 3 comments
Open

Nothing happens when I click [Map > Map Properties...] #3806

manake opened this issue Aug 12, 2023 · 3 comments
Labels
bug Broken behavior. usability Generally about making something more intuitive or efficient.

Comments

@manake
Copy link

manake commented Aug 12, 2023

Describe the bug
Nothing happens when I click [Map > Map Properties...].

To Reproduce
Steps to reproduce the behavior:

  1. Click on [Map > Map Properties...].

Expected behavior
Either it should be grayed out or something should happen.

Media

menu.mp4

Specifications:

  • OS: Windows 11
  • Tiled Version: 1.10.2
@manake manake added the bug Broken behavior. label Aug 12, 2023
@eishiya
Copy link
Contributor

eishiya commented Aug 12, 2023

The Map Properties were successfully opened. The reason it looks like nothing happened is because you already had them open.

Edit: These are the Map Properties, visible in your video:
image

@manake
Copy link
Author

manake commented Aug 12, 2023

Thanks, I realized this eventually too.

But in such case it should have a "tick" icon to indicate that it's already opened:

2

@eishiya
Copy link
Contributor

eishiya commented Aug 12, 2023

The Properties panel already has such a checkmark.
image
Such checkmarks are generally used for things that have a binary state such as on/off, which doesn't make sense for the content of the Properties panel, which can be in many states.

The Map Properties and Tileset Properties menu items are really only there because there's no good way to "select" Maps and Tilesets for viewing in the properties panel the way you can with Tiles, Objects, Layers, etc. #3101 is about making the Map and Tileset properties easier to find, and perhaps if a good solution is found there, these menu items could be removed entirely.

An alternative solution would be to briefly flash the panel, similar to how activating already activated windows works in some programs. I don't know whether this is possible in Qt.

@bjorn bjorn added the usability Generally about making something more intuitive or efficient. label Aug 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Broken behavior. usability Generally about making something more intuitive or efficient.
Projects
None yet
Development

No branches or pull requests

3 participants