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

VLC-refactor (CSS) Part 3 #869

Merged
merged 38 commits into from
May 4, 2021

Conversation

brenner-company
Copy link
Contributor

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

Om de PR's iets makkelijker/lichter te maken zal ik componenten groeperen in een aantal delen.


Volgende componenten zijn gerefactored (binnen deze PR):

  • panel-layout
  • printable-agenda-list
  • preview-container
  • publication-overview
  • publication-status-pill
  • radio-inline
  • scroll-wrapper
  • search
  • search-legend
  • side-nav
  • themes-selector
  • user
  • user-management-table

om VLC-componenten te refactoren en niet direct een verandering in functionaliteit/look-and-feel te veroorzaken heb ik nog enkele tijdelijke styling rules toegevoegd. Voor au-kaleidos-css kunnen die gevonden worden onder au-kaleidos-css/auk-additions.

…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
# Conflicts:
#	cypress/integration/unit/formally-ok.spec.js
@brenner-company
Copy link
Contributor Author

brenner-company commented May 3, 2021

Deze even als draft PR opgezet om de testen hiervoor te laten lopen. Als deze ok zijn publish ik dan ook de nieuwe versie van au-kaleidos-css package.

Daarna dus pas mergen. ⚠️

@brenner-company brenner-company marked this pull request as ready for review May 4, 2021 08:43
@brenner-company
Copy link
Contributor Author

Na de test die slagen gisteren en de au-kaleidos-css package die gepublished is, mag deze gemerged worden, maar wacht nog even op final go van @ValenberghsSven en/of @MikiDi.

Copy link
Contributor

@ValenberghsSven ValenberghsSven left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ziet er goed uit, testen idd groen

@ValenberghsSven ValenberghsSven merged commit 7c313d8 into development May 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants