Opened 7 years ago
Last modified 5 years ago
#4366 closed enhancement
[PATCH] Move barter UI to trade panel — at Version 1
Reported by: | elexis | Owned by: | |
---|---|---|---|
Priority: | Nice to Have | Milestone: | Alpha 22 |
Component: | UI & Simulation | Keywords: | patch |
Cc: | s0600204 | Patch: |
Description (last modified by )
Since #3934 0 A.D. supports adding and removing resources.
However the current barter panel has space for at most 4 resources. The 2 most recent commits in https://github.com/s0600204/0ad/commits/barterIconShift move the bartering UI from the bottom panel to the trade dialog, alleviating this issue:
On first sight it might seem like a usability penalty, but on second sight it doesn't. Currently, in order to barter one has to either recall where the market was built or recall how a market model looks and select it before being able to access the barter UI. With the patch, one can skip the first part, so it might be faster. On the other hand if one knows where a market was built (good practice: market at well known locations like besides CC), opening the trade dialog requires more mouse movement (distance in pixels). A hotkey to open the trade dialog would solve that issue again.
Change History (3)
by , 7 years ago
Attachment: | new_barter.jpg added |
---|
comment:1 by , 7 years ago
Description: | modified (diff) |
---|
by , 7 years ago
Attachment: | 4366_barterUI.patch added |
---|
Changes in one single patch, recently rebased (r19015, 36247e)