Skip to content

Commit b051f84

Browse files
authoredAug 27, 2023
Fix collection detail global css (#4143)
* Collection dependencies - reduce custom css, add missing space before helpertext moving CollectionUsedbyDependenciesList outside the pf-c-content section removes the need for custom pagination styling (we'll now get 2 different empty states for deps & rdeps when both are missing) No-Issue * Collection docs - rename css classes to hub-docs-{container,content,main,sidebar}, remove global toolbar margin affects the new insights breadcrumbs, this fixed it * hub-nav-title * duplicate/unused legacy role scss * hub-header-bordered: replace old header class * header-bottom -> hub-header-bottom * merge hub-toolbar and hub-list-toolbar css now that `.pf-c-toolbar__group { margin-left: 16px }` is limite to `.hub-docs-sidebar`, changing the left padding is no longer needed .. which makes hub-toolbar and hub-list-toolbar identical, merging * approvals: use ToolbarContent, add missing niceNames * hub-toolbar, hub-toolbar-left - unify from .toolbar, .hub-toolbar-wrapper
1 parent 9b0eca6 commit b051f84

File tree

52 files changed

+193
-369
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

52 files changed

+193
-369
lines changed
 

‎src/components/collection-dependencies-list/collection-dependencies-list.tsx

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { t } from '@lingui/macro';
2-
import { List, ListItem, ListVariant } from '@patternfly/react-core';
2+
import { List, ListItem } from '@patternfly/react-core';
33
import React from 'react';
44
import { Link } from 'react-router-dom';
55
import { CollectionVersion, CollectionVersionSearch } from 'src/api';
@@ -29,7 +29,7 @@ export const CollectionDependenciesList = ({
2929
}
3030

3131
return (
32-
<List variant={ListVariant.inline} className='hub-c-list-dependencies'>
32+
<List className='hub-c-list-dependencies'>
3333
{dependencies_repos.map((dependency, i) =>
3434
listDep(dependency, i, dependencies),
3535
)}
@@ -50,7 +50,7 @@ const listDep = (dependency, i, dependencies) => {
5050
} else {
5151
return (
5252
<ListItem key={i} style={{ marginRight: '70px' }}>
53-
{fqn}: {version_range}
53+
{fqn}: {version_range}{' '}
5454
<HelperText
5555
content={t`No version of ${fqn} exists that matches ${version_range}.`}
5656
/>
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,4 @@
1-
.hub-toolbar-wrapper {
2-
.toolbar {
3-
display: flex;
4-
justify-content: space-between;
5-
align-items: center;
6-
7-
.hub-pagination-container {
8-
display: flex;
9-
align-items: center;
10-
}
11-
}
1+
.hub-pagination-container {
2+
display: flex;
3+
align-items: center;
124
}

0 commit comments

Comments
 (0)
Please sign in to comment.