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

Eyedrop / Fill tool tweaks #790

Closed
plangworthy opened this issue Oct 7, 2014 · 4 comments
Closed

Eyedrop / Fill tool tweaks #790

plangworthy opened this issue Oct 7, 2014 · 4 comments
Labels
usability Generally about making something more intuitive or efficient.

Comments

@plangworthy
Copy link

It should be possible to use the eyedrop tool (right click) while the fill tool is selected. Right now the eyedrop tool only appears to be available when using the Stamp Brush.

Also in earlier versions of Tiled it was possible to eyedrop a null tile and then use the fill tool to fill an area with null tiles. It is already still possible to eyedrop a null tile and brush null tiles. I would like to request this feature be re-integrated.

@bjorn
Copy link
Member

bjorn commented Oct 7, 2014

Hmm, which earlier version of Tiled behaved like that? As far as I know, it was never possible to eye-drop a null tile. Removing tiles can only be done with the Eraser and the Rectangular Select tools. The reason for this is because it can be annoying if null tiles in the stamp would cause tiles to get erased.

Instead of allowing filling with null tiles, I think we need a magic wand tool for selecting regions, which can then be easily cut or deleted. Also, the eraser could be improved for example by allowing to quickly erase rectangular areas on right-click.

I agree eye-dropping should also be possible when the fill tool is selected.

@plangworthy
Copy link
Author

This was years and years ago, like version 5 or 6. It's possible I'm remembering wrong, or that it was actually just a bug.

I agree that wand selection and deleting would follow the graphics editor paradigm better that null filling. That would be completely acceptable as far as I'm concerned.

@bjorn
Copy link
Member

bjorn commented Oct 7, 2014

This was years and years ago, like version 5 or 6. It's possible I'm remembering wrong, or that it was actually just a bug.

You may be talking about the old Java version of Tiled then. It seems to allow you to do this for single tiles, after popping up the message "The selection is empty!" on trying to grab. :-)

I agree that wand selection and deleting would follow the graphics editor paradigm better that null filling. That would be completely acceptable as far as I'm concerned.

Alright. That's actually covered by issue #128.

@bjorn bjorn added the usability Generally about making something more intuitive or efficient. label Mar 7, 2017
@bjorn
Copy link
Member

bjorn commented Jun 3, 2017

Also requested on the forum. I'll try to get around to this soon.

@bjorn bjorn closed this as completed in 1c58205 Sep 5, 2017
@bjorn bjorn moved this to Completed in Roadmap Feb 3, 2023
@bjorn bjorn added this to Roadmap Feb 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
usability Generally about making something more intuitive or efficient.
Projects
Archived in project
Development

No branches or pull requests

2 participants