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

Suggestions for context menu #7

Open
Ablu opened this issue Apr 20, 2011 · 4 comments
Open

Suggestions for context menu #7

Ablu opened this issue Apr 20, 2011 · 4 comments
Labels

Comments

@Ablu
Copy link
Contributor

@Ablu Ablu commented Apr 20, 2011

  • Remove this tile from map
  • Replace all tiles of a kind
  • Select all tiles of a kind
@bjorn

This comment has been minimized.

Copy link
Owner

@bjorn bjorn commented Apr 20, 2011

The "replace all tiles of a kind" is covered by the "select all tiles of a kind" + "fill tool" (when holding shift it fills the selection). As a context menu item it also has the problem of selecting with which tile to replace it. The other two sound good.

@stefanbeller

This comment has been minimized.

Copy link
Contributor

@stefanbeller stefanbeller commented Jul 16, 2011

Ok I started implementing that.
Now I have problems with tiles and cells, since you can only select tiles, but at a map there are cells
(tiles with flipped=yes/no attribute) .

My idea would be to make it a menu entry in the layer menu and to use the current stamp.

That would mean this way of handling:

  • select all tiles you want to replace/remove and have them in your stamp
  • LayerMenu->"Select all tiles of these"
    -> Now this behaves like the selection tool: A selection will be setup, at which the region will cover all the
    cells within the selection
  • Within that selection you can either use the erase tool or the stampbrush tool to replace/remove all the tiles in the region.
@bjorn

This comment has been minimized.

Copy link
Owner

@bjorn bjorn commented Jul 16, 2011

My idea would be to simply ignore the flipped state of the cells for the tileset right-click menu. We can have a different approach for the map, probably an additional tile selection tool that selects all cells that are the same as the clicked one. Similar to the Select by Color Tool in GIMP.

@stefanbeller

This comment has been minimized.

Copy link
Contributor

@stefanbeller stefanbeller commented Jul 16, 2011

I think, that this is an inconsistent approach.
First we should do care for the flipped state, since every other tool does.
(Second I would not put it into a rightclickmenu, since all other tools are either in toolbars or in the menu)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.