Skip to content
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

fix: toppjuster tabellinnhold som standard #2485

Merged
merged 6 commits into from
Nov 17, 2021
Merged

Conversation

wkillerud
Copy link
Contributor

ISSUES CLOSED: #2434

☑️ Sjekkliste

  • Jeg har lest CONTRIBUTING og dokumentasjon det henvises til
  • Jeg har satt target branch til main, eller external-contributions dersom pull requesten kommer fra en fork
  • Jeg har kjørt yarn build og yarn ci:test og disse gir ingen feil
  • Jeg har lagt til tester som demonstrerer at feilen er rettet eller featuren fungerer
  • Jeg har skrevet relevant dokumentasjon

@wkillerud wkillerud self-assigned this Nov 10, 2021
@wkillerud wkillerud added the ✨ Forslag Forslag til nye funksjoner og endringer label Nov 10, 2021
piofinn
piofinn previously approved these changes Nov 10, 2021
@piofinn
Copy link
Contributor

piofinn commented Nov 10, 2021

Ville kanskje også vurdert å legge til dette (eller tilsvarende) under "Rader med handling" på portalsiden:

Knapper skal komme til slutt i raden og høyrejusteres. Tekstinnholdet i rader med handling bør sentreres vertikalt slik at det kommer på linje med knappene, med mindre det er mye innhold som bryter over flere linjer. I listevisning skal knappene venstrejusteres og vises under en egen overskrift.

@wkillerud
Copy link
Contributor Author

Ja, kom på her at jeg glemte oppdatere teksten under Typografi også, så der står det fremdeles at vi skal sentrere. Oppdaterer når Cypress gir oss nye snapshots.

@piofinn
Copy link
Contributor

piofinn commented Nov 10, 2021

Oppdaterer når Cypress gir oss nye snapshots.

Topp! Jeg ønsker meg fortsatt også et generelt eksempel "above the fold" på portalsiden, forresten 😅 Hva med noe som dette (beklager wall of text 😑), som kan plasseres rett etter ingressen?

import { ExampleComponentProps } from "doc-utils";
import React, { VFC } from "react";
import { Table, TableBody, TableCaption, TableCell, TableHead, TableHeader, TableRow } from "../src";

const columns = ["Dato", "Saksnummer", "Kundenummer", "Kundenavn", "Milepæl", "Følger saken"];

const rows = [
    ["24.02.2020", "20-1234567", "010203 99887", "Ola Nordmann", "Opprettet", "Siri Saksbehandler"],
    ["13.04.2019", "20-8382811", "010203 99887", "Kari Nordkvinne", "Opprettet", "Siri Saksbehandler"],
    ["31.07.2017", "20-1111", "010203 99887", "Kari Nordkvinne", "Opprettet", "Per Persen"],
];

const TableExample: VFC<ExampleComponentProps> = ({ boolValues }) => {
    return (
        <Table compact={boolValues?.["Kompakt"]} fullWidth>
            <TableCaption srOnly>Overskrift for skjermlesere</TableCaption>
            <TableHead>
                <TableRow>
                    {columns.map((column, index) => (
                        <TableHeader key={index}>{column}</TableHeader>
                    ))}
                </TableRow>
            </TableHead>
            <TableBody>
                {rows.map((row, rowIndex) => (
                    <TableRow key={rowIndex}>
                        {row.map((cell, cellIndex) => (
                            <TableCell key={cellIndex}>{cell}</TableCell>
                        ))}
                    </TableRow>
                ))}
            </TableBody>
        </Table>
    );
};

export default TableExample;

export const tableExampleCode = ({ boolValues }: ExampleComponentProps): string => `
const columnHeaders = ["Dato", "Saksnummer", "Kundenummer", "Kundenavn", "Milepæl", "Følger saken"];

const rows = [
    ["24.02.2020", "20-1234567", "010203 99887", "Ola Nordmann", "Opprettet", "Siri Saksbehandler"],
    ["13.04.2019", "20-8382811", "010203 99887", "Kari Nordkvinne", "Opprettet", "Siri Saksbehandler"],
    ["31.07.2017", "20-1111", "010203 99887", "Kari Nordkvinne", "Opprettet", "Per Persen"],
];

<Table compact={${boolValues?.["Kompakt"]}} fullWidth>
    <TableCaption srOnly>Overskrift for skjermlesere</TableCaption>
    <TableHead>
        <TableRow>
            {columnHeaders.map((header, index) => (
                <TableHeader key={index}>
                    {header}
                </TableHeader>
            ))}
        </TableRow>
    </TableHead>
    <TableBody>
        {rows.map((row, rowIndex) => (
            <TableRow key={rowIndex}>
                {row.map((cell, cellIndex) => (
                    <TableCell key={cellIndex}>{cell}</TableCell>
                ))}
            </TableRow>
        ))}
    </TableBody>
</Table>
`;

wkillerud and others added 4 commits November 11, 2021 09:49
affects: @fremtind/jkl-table-react, @fremtind/jkl-table

`verticalAlign="top"` er nå en no-op og kan fjernes

ISSUES CLOSED: #2434
affects: @fremtind/jkl-table-react
affects: @fremtind/jkl-table-react
@wkillerud wkillerud force-pushed the fix/change-default-valign branch 2 times, most recently from 92ed8a6 to 489b450 Compare November 11, 2021 09:32
@wkillerud wkillerud requested a review from piofinn November 11, 2021 09:59
affects: @fremtind/jkl-table-react
@wkillerud wkillerud force-pushed the fix/change-default-valign branch from 489b450 to 9dbd63d Compare November 12, 2021 13:16
@wkillerud
Copy link
Contributor Author

Har lagt til en "eksempelvelger" i toppen etter ønske i #2333

@wkillerud wkillerud merged commit 779105a into main Nov 17, 2021
@wkillerud wkillerud deleted the fix/change-default-valign branch November 17, 2021 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Forslag Forslag til nye funksjoner og endringer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Innhold i tabeller må toppjusteres som default
3 participants