Skip to content

Commit 195ec27

Browse files
authored
Merge pull request #263 from devtron-labs/fix/pagination-ui
fix: border in page number button in pagination
2 parents 6e8fa9b + 8731170 commit 195ec27

File tree

4 files changed

+4
-13
lines changed

4 files changed

+4
-13
lines changed

package-lock.json

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@devtron-labs/devtron-fe-common-lib",
3-
"version": "0.2.15",
3+
"version": "0.2.16",
44
"description": "Supporting common component library",
55
"type": "module",
66
"main": "dist/index.js",

src/Common/Pagination/Pagination.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,6 @@ const Pagination = ({
8181
const pageNoIndex = pages.findIndex((page) => page.selected)
8282
const visiblePages = pages.filter((page) => page.isVisible)
8383
return (
84-
// TODO: Remove pagination-wrapper from dashboard
8584
<div className={rootClassName || ''}>
8685
<div className="page-number">
8786
{offset + 1} - {lastPageNo} of {size}

src/Common/Pagination/pagination.scss

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -14,15 +14,6 @@
1414
* limitations under the License.
1515
*/
1616

17-
.pagination-wrapper {
18-
display: flex;
19-
align-items: center;
20-
padding: 0px 24px;
21-
justify-content: space-between;
22-
margin: 0px;
23-
background-color: var(--white);
24-
}
25-
2617
.pagination {
2718
margin-top: 16px;
2819
margin-bottom: 16px;
@@ -99,6 +90,7 @@
9990
background-color: var(--white);
10091
padding: 0px;
10192
user-select: none;
93+
border: none;
10294
}
10395

10496
.page__button--icon {

0 commit comments

Comments
 (0)