Table update #29
Merged
Table update #29
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
OliEfr
approved these changes
May 23, 2021
OliEfr
reviewed
May 24, 2021
| // THIS IS MODIFIED LINE BY OLIVER HAUSDOERFER TO ALWAYS SHOW SORT ICONS | ||
| // var vsThvue_type_template_id_6008fdf0_render = function () { var _vm = this; var _h = _vm.$createElement; var _c = _vm._self._c || _h; return _c('th', { attrs: { "colspan": "1", "rowspan": "1" }, on: { "click": function ($event) { return _vm.sortValue() } } }, [_c('div', { staticClass: "vs-table-text" }, [_vm._t("default"), (true) ? _c('span', { staticClass: "sort-th" }, [_c('vs-icon', { staticClass: "th-sort-icon", attrs: { "icon": _vm.currentSort == 1 ? 'expand_less' : 'expand_more' } })], 1) : _vm._e()], 2)]) } | ||
|
|
||
| // THIS IS THE FURTHER MODIFIED LINE BY LENNART GUDERIAN TO SHOW AN UNFOLD_MORE-ICON IN THE UNSORTED AND THE |
There was a problem hiding this comment.
@Noxdor Haha gewöhne dir bitte nicht alle meine schlechten collaborations habits an 👍 😆
Aber die Lösung wie es jetzt ist finde ich gut :)
|
😂 Ich will es doch nur einheitlich halten 😝 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Die visuellen Updates sind fertig! Die funktionellen kommen als nächstes, die werden wohl noch ein bisschen dauern :)
Besonders folgende in #26 gesammelten Verbesserungsvorschläge wurden eingefügt:
a. Die Farbe Rot wurde aufgrund ihrer negativen Bedeutung entfernt und durch ein grau ersetzt (Eine Veränderung der Ansicht nach im Zweit- oder Drittversuch bestandener Prüfung folgt noch.)
b. Die Farbthema wurde stärker an die Graustufenlogik der Originaltabelle angepasst, ohne zu langweilig zu werden.
c. Die einheitliche Farbe der ersten Spalte wurde entfernt.
a. Achtung! Diese Neuerung repariert lediglich das visuelle Problem. Allgemeine Probleme der Sortierung wurden hier noch nicht gelöst, dafür muss die dahinterliegende Datenstruktur von einem mehrdimensionalen Array auf ein Array mit Objekten für jede Zeile mit entsprechenden Attributen umgestellt werden. Diese Verbesserung folgt noch.
Insbesondere fehlen noch folgende Anpassungen: Filter und Datenstruktur - Optimierung, Aufklappbare Module, An- und abwählbare Spalten. Außerdem ist ist das Problem mit dem Brave Browser noch nicht gelöst, beziehungsweise noch nicht mal angeschaut bisher :D Also da wartet noch eine gute Portion Programmierspaß für die kommenden Wochen und Monate, je nachdem, wie die Zeit bleibt :D
Ganz lieben Dank besonders an @OliEfr für das Sammeln des Feedbacks und den persönlichen Input, sowie an @C0ntroller fürs aufmerksam machen auf das Problem mit dem Brave Browser und für das Ausreden der Gradienteneffekte :D