Feature/kas 2219 headers publicaties grid #868
Conversation
…19-Headers-publicaties-grid
Toevoeging korte headers Datatable naar native html table Aanpassing volgorde van kolommen in grid
…grid # Conflicts: # app/pods/publications/index/route.js # app/pods/publications/index/template.hbs
|
Ik ben niet helemaal mee welke volgorde van kolommen er hier nu gevolgd is. Er is enerzijds de volgorde die OVRB doorgegeven heeft (zie ook comment in Jira ticket van 11 maart)
Anderzijds de volgorde uit het nieuwe design in https://www.figma.com/file/D8NJqS92aZbzSx8GeSOaDr/%F0%9F%92%AF-Kaleidos---Designs-Digitale-Publicaties-v3?node-id=2688%3A54 Maar deze branch lijkt geen van beide te volgen? |
|
Ik heb de volgorde gevolgd van de designs. Daarbij had ik dan 1 dingetje over het hoofd gezien. |
|
VLC css is Karel zoveel mogelijk aan het verwijderen, uiteindelijk willen we daar vanaf, dus nieuwe toevoegen lijkt dan ook niet gewenst |
…19-Headers-publicaties-grid # Conflicts: # app/styles/custom-components/_vlc-publication-overview.scss
…n/kaleidos-frontend into feature/KAS-2219-Headers-publicaties-grid
| .auk-publication-table{ | ||
| max-height: 80%; | ||
| } |
There was a problem hiding this comment.
Deze auk-publication-table
dat zorgt voor een onnodige whitespace onder de footer
zonder die klasse plakt de footer tegen de bodem van de pagina, maar op zich niet zo erg want in een volgend ticket gaat die footer aangepast worden en gaat de auk-pagination css ervoor zorgen dat die footer juist is
| </th> | ||
| {{else}} | ||
| {{#if column.sortable}} | ||
| <div class="auk-scroll-wrapper auk-publication-table"> |
There was a problem hiding this comment.
die auk-publications-table, waarom staat die hier?
In de design/prototype is de height 100%
De klikbaarheid van de rij weghalen vind ik persoonlijk een slechte keuze. |
|
@erikap |
|
@ValenberghsSven Mijn opmerking dat de row niet klikbaar mag zijn, komt uit het prototype en een discussie rond klikbare table rows met @Wolfr in de ember-data-table addon. Een hele rij klikbaar maken is niet accessible en wordt niet correct geïnterpreteerd door screen readers. |
|
We vinden dit voor de UX wel handig en hebben in laatste prototype een versie geïmplementeerd die wel toegankelijk is. Heb het na die discussie in ember-data-table beter onderzocht, blog post geschreven en Fanny heeft techniek toegepast.
https://johanronsse.be/2021/02/27/accessible-clickable-table-rows/
http://development.kaleidos-prototype.mono.digital/shared/publications/overview.html
…Sent from my iPhone
On 8 May 2021, at 15:14, Erika Pauwels ***@***.***> wrote:
@ValenberghsSven Mijn opmerking dat de row niet klikbaar mag zijn, komt uit het prototype en een discussie rond klikbare table rows met @Wolfr in de ember-data-table addon. Een hele rij klikbaar maken is niet accessible en wordt niet correct geïnterpreteerd door screen readers.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
…19-Headers-publicaties-grid # Conflicts: # app/pods/publications/index/template.hbs
app/pods/publications/index/route.js
Outdated
| @@ -34,7 +34,11 @@ export default class PublicationsIndexRoute extends Route { | |||
| async model(params) { | |||
| const statusIds = []; | |||
| let ministerFilter = {}; | |||
|
|
|||
| this.params = params; | |||
There was a problem hiding this comment.
Dit lijkt me niet meer nodig nu alles weer in 1 method zit
| @tracked page = 0; | ||
| @tracked size = 10; |
There was a problem hiding this comment.
Van @MikiDi geleerd dat er een issue is met @tracked query params in de Ember versie die we momenteel gebruiken. Voor query params gebruiken we dus nog de oude Ember-manier: geen @tracked en wijzigen via set('page', 5) uit @ember/object
Bedankt voor de input @Wolfr 👍 @ValenberghsSven @TaneeG Aangezien het vroeger een expliciete vraag was om die rows clickbaar te maken, stel ik voor het in deze PR te doen zoals voorheen. Dan zet ik een nieuwe story op de backlog om het te corrigeren naar de nieuwe, accessible manier die @Wolfr beschrijft in zijn blogpost. |
delete global variable
|
Om even een freeze te maken in de designs voor deze story. Deze screenshot is de gebruikte volgorde |
De originele tabel is naar een native tabel veranderd. Ik heb de paginatie placeholder gewijs toegevoegd, want deze werkt niet meer. Zoals met Michael besproken komt deze terug via een ander ticket.
De Inhoud van de rijen is niet voor dit ticket om op te lossen
Voor de designs => https://www.figma.com/file/D8NJqS92aZbzSx8GeSOaDr/%F0%9F%92%AF-Kaleidos---Designs-Digitale-Publicaties-v3?node-id=2688%3A54