-
Notifications
You must be signed in to change notification settings - Fork 27
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
component/legislation-introduction #192
component/legislation-introduction #192
Conversation
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.
Looks good. Have a few minor asks.
The addition to package-lock.json seems innocuous, but why was it added? I don't see any changes to package.json
<b className="mzp-u-body-sm">{matterStatus.matter?.title}</b> | ||
<div |
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 don't think mzp-u-body-sm
is valid Mozilla protocol CSS class
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.
where can I find a list of valid classes? I use their website but in a case like this where I want something that's just a little bit different from the listed items, it becomes very difficult to figure out what is valid and what is not.
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 sure if there's a page that lists them. I didn't find the class inside node_modules/@mozilla-protocol/core/css
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.
https://protocol.mozilla.org/docs/naming.html
Here's how they name their class though
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.
Makes sense to me! Sorry that the matter index pipeline isn't done yet.
Link to Relevant Issue
This pull request resolves #172
Storybook
See the component [here](https://brianl3.github.io/cdp-frontend/?path=/story/library-details-legislation-introduction--approval-story