-
Notifications
You must be signed in to change notification settings - Fork 78
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
feat: update fundamental-styles to v0.12.0-rc.18 #1172
Conversation
Deploy preview for fundamental-react ready! Built with commit 8f48459 |
Re: treeview - let's just delete it. |
import React, { useState, useEffect } from 'react'; | ||
import Button from '../../../src/Button/Button'; | ||
|
||
const CopyButton = ({ copyText }) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👏
return ( | ||
<> | ||
<Header /> | ||
<Title /> | ||
<Toc /> | ||
<Subtitle /> | ||
{context?.parameters?.deprecated && <Description desc={context?.parameters?.deprecated} />} | ||
{!context?.parameters?.noImport && showImport()} | ||
{context?.parameters?.displaySubComponentImports && showSubImports()} | ||
{context?.parameters?.description && <Description desc={context?.parameters?.description} />} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not caused by this pr (it's in master too) - but our descriptions are gone 😢
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can see the description for our Tree component here: https://deploy-preview-1172--fundamental-react.netlify.app/?path=/docs/component-api-tree--wrapping-and-non-wrapping-contents
Edit: I just realized what you meant. I know how to fix this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I got the story descriptions working here, but not the component descriptions: #1189
# Conflicts: # package.json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huge push 👏 been mainly testing the newer components, and haven't noticed anything.
Agree to also removing TreeView
if it should be done in this PR. The docs page is pretty broken.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huge lift! 💪
Closing and reopening to trigger checks. |
Description
BREAKING CHANGE:
In this change we
fundamental-styles
to0.12.0-rc.18
and support the new features and fixes therein.LayoutGrid
component in favor of the new grid system based onContainer
,Row
, andColumn
componentsLayoutPanel.colSpan
property. Added story forLayoutPanel
in the new grid system.TreeView
component in favor of the newTree
component.Question
TreeView
styles are broken and I've marked it as deprecated. Do we want to remove it entirely instead?Yes
Fixes #646 #310