Problem/Motivation

https://www.thedroptimes.com/43519/alejandro-madrigal-leiva-releases-fir...

Would be good to see how this looks.

Steps to reproduce

Proposed resolution

Set up using dev guide and install SDC Styleguide and test it!

Remaining tasks

  1. Test SDC Styleguide with SDDS components
  2. Decide if we should add to our docs or something

User interface changes

API changes

Data model changes

Comments

kristen pol created an issue. See original summary.

kristen pol’s picture

Issue summary: View changes

Update summary to link to dev guide.

pdureau’s picture

Hello Kristen,

Since August, we are working on filling the gap between sdc_styleguide & ui_patterns 2 stories formats: #3464909: Collaboration with UI Patterns 2

On UI Patterns 2 side, we plan to deliver the changes in our last beta (beta5, early November)

On SDC Styleguide side, there are 2 main changes expected:

  • the slots are not compliant with the SDC render element because they are accepting markup string instead of render arrays (SDC render element accept only plain text scalars in slots)
  • the filename structure may not be compatible with plugin managers YML discoveries (sdc_styleguide is not using a plugin manager but a custom discovery service, but UI Patterns 2 will do, and I believe this will be the expected way if we move the format to core one day)

So, I would not advice to adopt the current (1.1) format of sdc_styleguide. Maybe a future release.

kristen pol’s picture

Thanks for the details! I’ll move this to postponed for now.

kristen pol’s picture

Status: Active » Postponed

Forgot to update status

pdureau’s picture

Hi Kristen,

Since October:

  • I am getting in touch with sdc_styleguide maintainer from time to time, adopting the shared story format seems still planned but no date shared yet.
  • UI Patterns 2.0.0 was released in February with great traction, being the first stable module using the story format
  • steepbase module has also adopted this format in January, and I hope more will join this year.
kristen pol’s picture

Thanks for the update :)