Opened 12 years ago
Closed 12 years ago
#1122 closed enhancement (fixed)
[PATCH] Add terrain eyedropper tool / show current texture
Reported by: | historic_bruno | Owned by: | ben |
---|---|---|---|
Priority: | Nice to Have | Milestone: | Alpha 10 |
Component: | Atlas editor | Keywords: | patch |
Cc: | Patch: |
Description
Patch to add an eyedropper tool to Atlas, like what most paint programs have. It samples the terrain texture under the brush and makes it the currently selected texture. It's very handy for expanding texture areas. The patch also adds a box in the terrain panel to show current brush texture.
Attachments (2)
Change History (10)
by , 12 years ago
Attachment: | atlas_eyedropper-01182012.patch added |
---|
comment:1 by , 12 years ago
comment:2 by , 12 years ago
Milestone: | Alpha 9 → Alpha 10 |
---|
follow-up: 4 comment:3 by , 12 years ago
Does the eye-dropper need to be a separate tool, rather than being something like right-click or shift-left-click with the normal 'paint' tool? The latter sounds more efficient since users will probably want to frequently switch between picking and painting textures, when working on a varied piece of land.
In LoadPreview
, if you hit the "invalid/missing texture" case then you'll exit from the function and leave the panel in a frozen state. Probably should call Thaw()
before return
to ensure the freezedness state stays balanced.
comment:4 by , 12 years ago
Replying to Philip:
Does the eye-dropper need to be a separate tool, rather than being something like right-click or shift-left-click with the normal 'paint' tool? The latter sounds more efficient since users will probably want to frequently switch between picking and painting textures, when working on a varied piece of land.
Good point, it should be as convenient as possible. Right click is not viable, since that's used for lower priority painting. I'll ask Michael what he would prefer since he does most of the map making :)
by , 12 years ago
Attachment: | atlas_eyedropper-03202012.patch added |
---|
comment:5 by , 12 years ago
New patch:
- The eyedropper is combined with the Paint tool (but not yet Replace or Fill - could be added later).
Shift
key is used to activate eyedropper.- At first I tried combining the eyedropper with the painting states, but IMO it was too easy to accidentally paint terrain by releasing the
Shift
key while moving the mouse. I thought it's more intuitive to give the eyedropper its own state, so the states are like this:- Waiting - if
Shift
key pressed, enter the Eyedropper state, else if mouse button pressed, enter the Paint states. - Paint - if mouse button released, enter the Waiting state.
- Eyedropper - if
Shift
key released, enter the Waiting state.
- Waiting - if
- The eyedropper has its owns fixed-size brush separate from the paint brush, otherwise it's not clear which texture will be selected.
- Fixed the missing
Thaw()
.
comment:6 by , 12 years ago
Keywords: | patch added |
---|
comment:7 by , 12 years ago
Keywords: | review removed |
---|
Eyedropper in action: http://i.imgur.com/Sik6D.jpg