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

Create "Tax Detail" view in Orders dashboard #1857

Closed
brent-hoover opened this issue Feb 17, 2017 · 14 comments
Closed

Create "Tax Detail" view in Orders dashboard #1857

brent-hoover opened this issue Feb 17, 2017 · 14 comments
Assignees
Milestone

Comments

@brent-hoover
Copy link
Collaborator

brent-hoover commented Feb 17, 2017

Related to #1793

We need to have a view where it can display the taxes charged per line item. There will probably need to be an "Inventory Detail" view and others so we may want to create a UI around that.

@joykare
Copy link
Contributor

joykare commented Feb 27, 2017

@rymorgan Some design help on this would be awesome

@rymorgan
Copy link
Contributor

rymorgan commented Mar 1, 2017

closed-state

@rymorgan
Copy link
Contributor

rymorgan commented Mar 1, 2017

open-state

@rymorgan
Copy link
Contributor

rymorgan commented Mar 1, 2017

line-items

@rymorgan
Copy link
Contributor

rymorgan commented Mar 1, 2017

@joykare Let me know if you have questions. There are some other clean up items in here that I have been working on too w/ the layout of the type left right. Etc. I want the items to rollover to light grey onhover so that the user knows they are clickable. This action can mimic what happens to lists.

@joykare
Copy link
Contributor

joykare commented Mar 1, 2017

@rymorgan cool will hit you up.

@joykare joykare added in progress and removed ready labels Mar 1, 2017
@brent-hoover
Copy link
Collaborator Author

@joykare Can I get an update on the status of this?

@joykare
Copy link
Contributor

joykare commented Mar 6, 2017

@zenweasel currently working on it. Its not yet complete.

@rymorgan
Copy link
Contributor

rymorgan commented Mar 9, 2017

invoice captured - no refund
invoice captured - refund applied

@rymorgan
Copy link
Contributor

rymorgan commented Mar 9, 2017

Here's my thinking. Captured total with refund field below it. Then you enter a refund amount, shows a spinner. Refund amount appears inline with the other line items. Feels much easier to read. Also, I'd like having the captured and adjusted totals next to each other for clarity for business users.

I've always thought it was weird sequencing to use the same button for refund as approve and capture because the other two actions are required to process and order but applying refund is not.

@rymorgan
Copy link
Contributor

rymorgan commented Mar 10, 2017

@joykare A few small updates to the UI on this.

  1. Added a bit more space between the items and the totals.
  2. Added a light grey bg behind the totals/taxes for more visual separation from the items.
  3. Moved the refund field above the totals, feels more intuitive this way.
  4. Removed the checkmarks.

Thanks!

invoice captured - no refund
refund

@joykare
Copy link
Contributor

joykare commented Mar 13, 2017

@rymorgan okay updating that

@kieckhafer kieckhafer modified the milestones: v1.0.0, v0.20.0 Mar 14, 2017
@joykare
Copy link
Contributor

joykare commented Mar 15, 2017

Related to #1849

@brent-hoover
Copy link
Collaborator Author

Closed via #2005

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

No branches or pull requests

4 participants