-
Notifications
You must be signed in to change notification settings - Fork 49
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
Update Bootstrap: CellLine Details #2059
Update Bootstrap: CellLine Details #2059
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.
Some minor fixes
<Button | ||
disabled={readOnly} | ||
size="sm" | ||
className="" |
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.
What happens when you remove className="" ?
Is it necessary?
</Panel> | ||
); | ||
} | ||
export default function EditModeRow({ |
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.
If you change this to a functional component.
Prefer to write it like this:
export default EditModeRow = ({ container, parent, element, readOnly }) => {
or you can write it like this:
const default EditModeRow = ({ container, parent, element, readOnly }) => {
and the export default EditModeRow
at the end of the file
Also consolidate EditModeHeader and OrderModeHeader into a single Header component.
61ea287
to
e4e53f9
Compare
* Remove unused method in CellLineDetails * Update CellLineDetails to new bootstrap version * Change cell line GeneralProperties PanelGroup to Accordion * Translate Form in GeneralProperties and CellLineName * Use utility classes in InvalidPropertyWarning * Use utility classes for Amount, separate logic and markup * Update AnalysesContainer button placement to utility classes * Update AnalysesContainer to new bootstrap version Also consolidate EditModeHeader and OrderModeHeader into a single Header component. * Remove unused css rule * Translate custom css to utility class * Style analysesTab Header using utility classes
* Remove unused method in CellLineDetails * Update CellLineDetails to new bootstrap version * Change cell line GeneralProperties PanelGroup to Accordion * Translate Form in GeneralProperties and CellLineName * Use utility classes in InvalidPropertyWarning * Use utility classes for Amount, separate logic and markup * Update AnalysesContainer button placement to utility classes * Update AnalysesContainer to new bootstrap version Also consolidate EditModeHeader and OrderModeHeader into a single Header component. * Remove unused css rule * Translate custom css to utility class * Style analysesTab Header using utility classes
No description provided.