-
Notifications
You must be signed in to change notification settings - Fork 98
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 inventory to patternfly-v4 #575
Update inventory to patternfly-v4 #575
Conversation
77d7b8f
to
c072532
Compare
packages/inventory/src/Inventory.js
Outdated
@@ -12,6 +12,8 @@ import { Provider } from 'react-redux'; | |||
import TagWithDialog from './TagWithDialog'; | |||
export const InventoryContext = createContext('inventory'); | |||
|
|||
console.log(React, 'Inventory version!'); |
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.
Console.log
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.
Nice catch!
c072532
to
89e05bb
Compare
89e05bb
to
aa94f22
Compare
} | ||
|
||
&:not(.ins-c-inventory__table--toolbar-has-items) { | ||
.pf-c-data-toolbar__content { |
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.
The data toolbar component no longer exists. These classNames should change to the toolbar.
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.
Huh, I thought I already fixed them...
@@ -1,7 +1,6 @@ | |||
module.exports = { | |||
externals: [ | |||
/^@patternfly\/.*/, | |||
/^@redhat-cloud-services\/.*/, |
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.
Is this intentional?
32d3c95
to
bdda585
Compare
b75b4f7
to
2540d0f
Compare
* Update inventory to patternfly-v4 * Remove data toolbar selectors * Wrap a-href element to div
* Update components pf4 (#558) * updated utils to PFv4 * Update PF dependencies versions * Upgrade components to PF4v4 * Update notifications to PF4v4 * Update source to PF4 v4 * Update rule components to PF4v4 (#565) * Update FEC remediations to PF4 * Update pdf generator PF dependencies (#566) * Change version to 2.1.0-beta * Updates css class from .pf-c-data-toolbar__item to .pf-c-toolbar__item primary-toolbar.scss DataToolbar was renamed to Toolbar, and the related css classes too * Update inventory to patternfly-v4 (#575) * Update inventory to patternfly-v4 * Remove data toolbar selectors * Wrap a-href element to div * Update general info to PF version 4 (#576) * Copy indexes after components are built (#579) * Upgrade PF of inventory insights to newest version * Upgrade inventory-compliance to Patternfly v4 (#574) * Final test updates to properly link packages (#582) Co-authored-by: Martin Maroši <marvusm.mmi@gmail.com> Co-authored-by: Richard Vsiansky <r.vsia@seznam.cz> Co-authored-by: PanSpagetka <rknaur@redhat.com> Co-authored-by: Josejulio Martínez <jmartine@redhat.com> Co-authored-by: Sebastian Gräßl <bastilian@users.noreply.github.com>
No description provided.