Skip to content

Swap equip and store on move popup#728

Merged
kyleshay merged 1 commit intodevfrom
equip-store
Aug 1, 2016
Merged

Swap equip and store on move popup#728
kyleshay merged 1 commit intodevfrom
equip-store

Conversation

@kyleshay
Copy link
Copy Markdown
Member

Really trivial, but figured make a PR to see what people think.

Figured we always have the equip column to the left of the inventory, the movepopup should be the same.

before:

screen shot 2016-07-29 at 11 59 24 am

after:

screen shot 2016-07-29 at 11 59 49 am

@SunburnedGoose
Copy link
Copy Markdown
Member

The only time this improves your mouse travel is you only equip one item. Otherwise, your mouse has to travel more if you're equipping and transferring items with all the repeat 'store' movements.

Splitting hairs, doesn't matter to me.

If we're on the subject, should we move the vault, or the current character closer to the mouse click? Where do you most often click?

@bhollis
Copy link
Copy Markdown
Contributor

bhollis commented Jul 31, 2016

Makes more sense to me - I like that the move popup now mirrors the overall layout. Not sure how I feel about the mouse-movement argument since I drag everything.

@kyleshay kyleshay merged commit ed0a53f into dev Aug 1, 2016
@kyleshay kyleshay deleted the equip-store branch August 1, 2016 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants