Quasar: [v0.15] QTable descending -->ascending (not working)

Created on 2 Nov 2017  路  5Comments  路  Source: quasarframework/quasar

Software version

Quasar:0.15
OS:Windows 7 & Linux
Node:6.11.5
NPM:4.1.2
Browsers:Chrome
iOS:N/A
Android:N/A
Any other software related to your bug:

What did you get as the error?

The QTable sorting is not working properly, we have to click 2 times instead of 1 time to invert the sorting order while going from descending to ascending.

What were you expecting?

Going from descending to ascending take only one click, not two.

What steps did you take, to get the error?

Using the test page I click on a sortable column one time, then click a second time. I repeat this step a couple of time and this is not working while going from descending to ascending.

Ordered ascending
image

I click ONE time on the same column and then nothing is reverted.
image

I digged a little bit into the code and sometime we get undefined for this.columnToSort on this line
https://github.com/quasarframework/quasar/blob/a2ea80f53fc6d9190b4b12803956860d00ed94e9/src/components/table/QTable.js#L104

Most helpful comment

Added binary-state-sort Boolean prop to QTable in Quasar v0.17.11 which enables 2 stages (ascending, descending) only.

All 5 comments

The user needs to also remove sorting. So sorting has 3 stages: none, ascending, descending. Clicking goes from one to the next one.

My fault, sorry. You're right, none is required to cancel the sort I had not thought about it.
Thank you for your time.

@rstoenescu

I disagree. At least I want to be able to restrict it to two stages. See Windows Explorer: Only two stages.

3 stages for sorting makes server side sorting more complicated.

Added binary-state-sort Boolean prop to QTable in Quasar v0.17.11 which enables 2 stages (ascending, descending) only.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

jean-moldovan picture jean-moldovan  路  3Comments

tombarfoot picture tombarfoot  路  3Comments

xereda picture xereda  路  3Comments

victorborgaco picture victorborgaco  路  3Comments

florensiuslaylim picture florensiuslaylim  路  3Comments