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

added sorter for taxonomy column #495

Merged
merged 1 commit into from Apr 23, 2016
Merged

added sorter for taxonomy column #495

merged 1 commit into from Apr 23, 2016

Conversation

@hjbflyer
Copy link
Contributor

@hjbflyer hjbflyer commented Apr 18, 2016

Hallo Andreas,

ich habe einen Vorschlag für einen weiteren Sorter.

@buchen
Copy link
Owner

@buchen buchen commented Apr 18, 2016

Die Taxonomy Column hat sogenannte Options, d.h. der Benutzer entscheidet ob er die komplette Klassifizierung sehen möchte, oder nur einen Ausschnitt (sagen wir "Industriezweig" der Branchenklassifizierung nach GIGS). Darum ist ein reine #toString auf die Klassifizierungen vielleicht etwas zu kurz.

Ich habe gerade mal mehrere Spalten einer Klassifikation einer Tabelle hinzugefügt. Da habe ich dann Probleme gehabt, dass nur die zweite Spalte sortiert wird. Ich bin mir auf gerade auch gar nicht sicher, wie das überhaupt funktioniert. Auch wenn ich mehrere "Kursänderung seit X" Spalten hinzufüge, werden die nicht korrekt sortiert obwohl ich die Option beachte... Wenn ich mir den Code ColumnViewerSorter anschaue, dann kann das auch nicht tun. Mehrere Spalten habe den selben Comparator.

Keine Ahnung was man da am besten macht. Entweder pro Option einen Comparator. Oder der ColumnViewerSorter muss noch checken ob die richtige Spalte selektiert wurde. Was immer sich auch eleganter anfühlt... 😄

Ich habe jetzt keine Zeit mir das vor Mittwoch (?) anzuschauen. Wenn Du Lust und Zeit hast, dann gerne eine Vorschlag.

@hjbflyer
Copy link
Contributor Author

@hjbflyer hjbflyer commented Apr 18, 2016

Habe mir das wohl zu einfach gemacht. Bei mir schien das alles gut zu funktionieren.

Bin leider auch bis nächste Woche nicht verfügbar.

@buchen
Copy link
Owner

@buchen buchen commented Apr 18, 2016

😄 Kein Problem, ich schaue es mir an. Ich merge wenn ich weiß wie ich das löse. Das Problem ist ja weniger dieser Change (welchen Text vergleichen) als das Optionen mit Comparator aktuell nicht tun (wegen dem Instanz Check).

@buchen buchen merged commit 936b9b0 into buchen:master Apr 23, 2016
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
buchen added a commit that referenced this pull request Apr 23, 2016
The taxonomy column can display the full classification, but also
individual levels (say: only 'sector' or only 'industry group').

Issue: #495
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants