Conversation
🎉 Snyk checks have passed. No issues have been found so far.✅ security/snyk check is complete. No issues have been found. (View Details) ✅ license/snyk check is complete. No issues have been found. (View Details) ✅ code/snyk check is complete. No issues have been found. (View Details) |
Coverage summary from CodacySee diff coverage on Codacy
Coverage variation details
Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: Diff coverage details
Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: See your quality gate settings Change summary preferencesFootnotes
|
| @@ -2,12 +2,19 @@ import { css } from '@linaria/core' | |||
| import { styled } from '@linaria/react' | |||
|
|
|||
| export const ElInputCheckboxContainer = styled.div` | |||
There was a problem hiding this comment.
note: some small scouts to the newly added InputCheckbox component.
| @@ -242,6 +244,33 @@ export const Alignment: Story = { | |||
| ], | |||
| } | |||
|
|
|||
There was a problem hiding this comment.
note: new "no padding" example for TableBodyCell
| @@ -27,11 +28,11 @@ const meta = { | |||
| children: { | |||
| control: 'select', | |||
| description: 'The row content.', | |||
There was a problem hiding this comment.
note: adding new canned children options that include checkboxes for the row. New stories demonstrating these are further down.
| @@ -25,6 +25,13 @@ export const elTableBodyRow = css` | |||
| min-height: var(--size-10); | |||
| max-height: var(--size-18); | |||
|
|
|||
There was a problem hiding this comment.
note: new selected styling. Take note of the code comment... we're doing the simplest thing that works for now.
| @@ -3,6 +3,7 @@ import { Menu } from '#src/core/menu' | |||
| import { TableBody } from './body' | |||
| import { TableBodyCell } from '../body-cell' | |||
| import { TableBodyRow } from '../body-row' | |||
There was a problem hiding this comment.
note: adding checkboxes to the TableBody docs
| /** | ||
| * A thin wrapper around [Input's checkbox implementation](./?path=/docs/core-input--docs) that is geared | ||
| * for use in tables built with [Table](./?path=/docs/core-table--docs). | ||
| */ |
There was a problem hiding this comment.
note: super thin wrapper around the newly added Input
| @@ -144,3 +146,30 @@ export const Alignment: Story = { | |||
| ), | |||
| ], | |||
| } | |||
There was a problem hiding this comment.
note: same new "no padding" support for TableHeaderCell as added to TableBodyCell
| @@ -6,7 +6,7 @@ import { StatusIndicator } from '#src/core/status-indicator' | |||
| import { TableCellPrimaryData } from './primary-data' | |||
| import { TagGroup } from '#src/core/tag-group' | |||
| import { Text } from '#src/core/text' | |||
There was a problem hiding this comment.
note: small scout
| @@ -1,12 +1,12 @@ | |||
| import { getNextSortDirection } from './sort-direction' | |||
| import { TableCellSortButton } from './sort-button' | |||
There was a problem hiding this comment.
note: small scout
| grid-auto-rows: auto; | ||
| grid-template-columns: var(--__table-columns); | ||
| justify-items: var(--__table-column-justification); | ||
| justify-content: var(--__table-column-justification); |
There was a problem hiding this comment.
note: small fix for broken style
Context
@reapit/elements/lab/table, but these don't facilitate the level of nuance required by the DS. For example:td,trand so on, but we may need the flexibility of using them in a div-based DOM structure instead.@reapit/elements/core/table.TableRowPrimaryAction#715, feat: Add newTableCellPrimaryData#716, feat: Add newTableCellDoubleLineLayout#717, feat: Add newTableBodyCell#718, feat: Add newTableBodyRow#719, feat: Add newTableBody#720 and feat: Add newTableRowMoreActions#721.TableCellSortButton#725, feat: Add newTableHeaderCell#726, feat: Add newTableHeaderRow#727, feat: Add newTableHead#728.This PR
TableCellCheckbox.TableBodyCellandTableHeaderCellto support the removal of their default padding.TableBodyRowto support "selected" styling.TableCheckboxTableBodyCellandTableHeaderCellTableBodyRowTableBody