Skip to content
This repository has been archived by the owner on Dec 13, 2020. It is now read-only.

Included tabs sorting #1590

Closed
metas-lc opened this issue Feb 13, 2018 · 2 comments
Closed

Included tabs sorting #1590

metas-lc opened this issue Feb 13, 2018 · 2 comments
Assignees
@siemiatj
Labels
branch:master branch:release priority:high type:bug
Milestone
Backlog

Comments

@metas-lc
Copy link

Is this a bug or feature request?

bug

What is the current behavior?

you can sort included tab only once and the arrow is not updating

Which are the steps to reproduce?

  1. open sales order, create a new one and add 2 lines
    (you can use this one https://w101.metasfresh.com:8443/window/143/1000425)
  2. select Line No. to sort
    => rows are sorted OK
    => arrow is still up NOK
  3. sort by Line No. again
    => nothing happens NOK

What is the expected or desired behavior?

each time you sort shall work
the arrow should update as well
@metas-lc metas-lc added this to the Backlog milestone Feb 13, 2018
siemiatj added a commit that referenced this issue Feb 14, 2018
@siemiatj
Fix sorting in table #1590
a4b0504
@siemiatj siemiatj mentioned this issue Feb 14, 2018
Merged
siemiatj added a commit that referenced this issue Feb 14, 2018
@siemiatj
Changes after review #1590
316fa1e
metas-mk added a commit to metasfresh/metasfresh that referenced this issue Feb 14, 2018
@metas-mk
Update ReleaseNotes.md
0f508fe 
Adding new Issue
metasfresh/metasfresh-webui-frontend-legacy#1590
Included tabs sorting
@metas-dh
Copy link
Member

Results of IT1
tested in mf15

sorting in included tabs:

  • rows are sorted: OK
  • arrow is updated correctly: OK
  • sorting works correctly as often as tried: OK

@metas-ts
Copy link
Member

cherry-picked to release

metas-ts pushed a commit that referenced this issue Feb 16, 2018
@siemiatj @metas-ts
Fix sorting in table #1590
42f2c98 
(cherry picked from commit a4b0504)
metas-ts pushed a commit that referenced this issue Feb 16, 2018
@siemiatj @metas-ts
Changes after review #1590
267aca4 
(cherry picked from commit 316fa1e)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@siemiatj siemiatj

Labels
branch:master branch:release priority:high type:bug
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
6 participants
@siemiatj @metas-ts @metas-mk @metas-lc @metasnw @metas-dh