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

Algemene todo: Refactor doel 2 - vlc naar auk #732

Open
41 of 61 tasks
Wolfr opened this issue Feb 17, 2021 · 2 comments
Open
41 of 61 tasks

Algemene todo: Refactor doel 2 - vlc naar auk #732

Wolfr opened this issue Feb 17, 2021 · 2 comments
Assignees

Comments

@Wolfr
Copy link
Contributor

Wolfr commented Feb 17, 2021

In navolging van #705 , is het erna kwestie van de .vlc- componenten te herschrijven naar .auk- componenten. Een groot deel hiervan is al gebeurd in het prototype, en daar hebben de componenten reeds een auk namespace.

Wat nog niet gebeurd is, is de vergelijking met productie. Dit zal veel makkelijker zijn dan de vl--componenten weg-refactoren, want deze vlc- code is nieuwer en de meeste code is initieel door Mono geschreven en is in het algemeen beter gedocumenteerd.

Dit is weer een invidiueel proces van case by case te bekijken hoe we aan de omzetting kunnen werken.

Heeft wellicht al auk-equivalent

Na te kijken

  • vlc-backlink
  • vlc-block-link
  • vlc-button-in-input
  • vlc-button-link
  • vlc-button-toolbar
  • vlc-date-range
  • vlc-document-card
    • Status: on-hold (auk-document-card is corresponderend component)
  • vlc-document-list (gebeurt werk aan: https://github.com/mono-company/kaleidos-prototype-v3/issues/88 )
    • Status: on-hold (auk-document-list is corresponderend component)
  • vlc-environment-pill - is de pill die toont in welke dev omgeving je zit, kan vereenvoudigd worden imo
    • Vraag: éénvoudiger qua stijl of qua code?
  • vlc-dropdown-menu
    • Vraag: momenteel geen (volledig uitgewerkt) alternatief voor?
  • vlc-input-field-block
  • vlc-list-section-header
    • Status: on-hold (auk-section-header is corresponderend component)
  • vlc-minister
  • vlc-mock-login Mock login #786
    • Status: on-hold (eerst uitzoeken of ember-mock-login aan te passen valt)
  • vlc-new-session
  • vlc-page-header
  • vlc-pagination
    • Status: on-hold (enkel gebruik binnen number-pagination component, die enkel binnen mock-login pagina voor komt & daar niet functioneert)
    • Vraag: in zijn geheel verwijderen?
  • vlc-panel-layout
  • vlc-printable-agenda-list
  • vlc-preview-container
  • vlc-procedure-step
    • Status: on-hold (auk-procedure-step is corresponderend component)
  • vlc-publication-overview
  • vlc-publication-status-pill
  • vlc-radio-inline
  • vlc-rdfa-editor
    • Status: on-hold
  • vlc-scroll-wrapper
  • vlc-search
  • vlc-search-legend
  • vlc-side-nav
  • vlc-status-timeline
    • Status: on-hold
  • vlc-subline
    • Status: on-hold
  • vlc-themes-selector
  • vlc-toaster-button
    • Status: on-hold
  • vlc-user
  • vlc-user-management-table
  • vlc-link-actions
  • vlc-login
  • vlc-color-badge-holder
  • vlc-color-badge
  • vlc-decision-result-select
    • Status: on-hold
  • vlc-numac-nummer-list
    • Status: on-hold

Ondertussen toegevoegd

  • vlc-publication-status-selector
  • vlc-subcase-description-grid

Agenda related

Zou ik niet mee beginnen, zie #788

  • vlc-agenda-detail-sidebar
  • vlc-agenda-detail-sidebar-item
  • vlc-agenda-detail-sidebar-section-header
  • vlc-agenda-items-section-header
  • vlc-agenda-items
  • vlc-agenda-meta
@Wolfr
Copy link
Contributor Author

Wolfr commented Mar 31, 2021

Deze issue is eigenlijk een beetje verkeerd geformuleerd. Ik herschrijf.

brenner-company added a commit that referenced this issue Apr 15, 2021
…/732-vlc-refactor/button

# Conflicts:
#	app/styles/app.scss
#	app/styles/au-kaleidos-css/_auk-u-generic.scss
brenner-company added a commit that referenced this issue Apr 15, 2021
…/732-vlc-refactor/key-value

# Conflicts:
#	app/styles/app.scss
brenner-company added a commit that referenced this issue Apr 16, 2021
@brenner-company
Copy link
Contributor

brenner-company commented Apr 16, 2021

Opmerking mbt refactoring per component zal ik hieronder wat onderhouden.

Box

(geen opmerkingen)

Button

  • Aangezien 'button-in-input' hier ook mee overlapte is deze tegelijkertijd mee gerefactored.

Container

Input

(geen opmerkingen)

Key-value

  • Hierna moet ook nog even auk key value #769 verder opgevolgd worden om alle bestaande (Ember) component instanties terug om te zetten naar HTM/CSS en dit ook aan te passen binnen de styling (mbt toekomstig gebruik).

Hr

  • Zou geen slecht idee zijn om hiervoor ook nog even een entry aan te maken binnen de styleguide.

Navbar

(geen opmerkingen)

Panel

(geen opmerkingen)

Pill

(geen opmerkingen)

Tabs + Tabs-reverse

  • Aangezien deze 2 versies binnen au-kaleidos-css onder dezelfde noemer gerekend worden, zijn deze tegelijkertijd gerefactored.

Toolbar

Backlink

(geen opmerkingen)

Block-link

(geen opmerkingen)

Button-in-input

(geen opmerkingen)

Button-link

(geen opmerkingen)

Button-toolbar

(geen opmerkingen)

Date-range

(geen opmerkingen)

Input-field-block

  • Door het iets hogere aantal instanties en de mogelijke visuele impact op elementen (binnen en buiten) is hier toch meer tijd in gekropen dan eerst gedacht.

Minister

(geen opmerkingen)

New-session

  • Gerefactored met bestaande auk- utility/helper classes (enkel .auk-u-text-condensed-line-height moeten toevoegen, maar zou die nog eens afgechecked moeten worden of dit, in deze vorm, wel ok is).

Page-header

(geen opmerkingen)

Panel-layout

  • Refactor met heel wat impact op omliggende elementen/structuren waardoor nu al een deel van vlc-side-nav & vlc-scroll-wrapper-refactor hier in mee genomen is.

Printable-agenda-list

  • Geen makkelijke refactor aangezien hier de styling ook grotendeels 'broken' was (raar dat hier nooit een issue voor aangemaakt is). Ik heb deze wat in orde gebracht, maar zou effectief nog eens afgetoetst moeten worden aan een design.

Preview-container

(geen opmerkingen)

Publication-overview

  • De overblijvende (gerefactored) styling in combinatie met styles/custom-utilities/_autocomplete.scss is nog steeds iets raar en lijkt meer op tijdelijke styling dan iets anders. Deze zou op termijn er nog zeker uit moeten gaan.

brenner-company added a commit that referenced this issue Apr 21, 2021
brenner-company added a commit that referenced this issue Apr 21, 2021
brenner-company added a commit that referenced this issue Apr 27, 2021
# Conflicts:
#	app/components/sessions/session-agenda-nav.hbs
brenner-company added a commit that referenced this issue Apr 27, 2021
ValenberghsSven added a commit that referenced this issue Apr 29, 2021
brenner-company added a commit that referenced this issue Apr 30, 2021
…us-connected' into DES/enhancement/#732-vlc-refactor

# Conflicts:
#	app/components/publications/publication-sidebar.hbs
#	app/pods/agenda/documents/template.hbs
#	app/styles/au-kaleidos-css/_auk-scroll-wrapper.scss
#	app/styles/au-kaleidos-css/_auk-sidebar.scss
#	app/styles/au-kaleidos-css/_auk-tabs.scss
#	app/styles/au-kaleidos-css/_s-colors.scss
brenner-company added a commit that referenced this issue Apr 30, 2021
# Conflicts:
#	cypress/integration/unit/formally-ok.spec.js
brenner-company added a commit that referenced this issue Apr 30, 2021
ValenberghsSven added a commit that referenced this issue May 4, 2021
brenner-company added a commit that referenced this issue May 4, 2021
brenner-company added a commit that referenced this issue May 4, 2021
…ement/#732-vlc-refactor

# Conflicts:
#	app/styles/app.scss
brenner-company added a commit that referenced this issue May 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants