Skip to content

Commit

Permalink
Merge pull request #238 from appwrite/refactor/change-table-headers
Browse files Browse the repository at this point in the history
refactor: remove eyebrow headings in document tables
  • Loading branch information
TorstenDittmann authored Jan 27, 2023
2 parents aea8c5f + 508fb6e commit 6fca232
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
3 changes: 2 additions & 1 deletion src/lib/elements/table/cellHead.svelte
Original file line number Diff line number Diff line change
@@ -1,14 +1,15 @@
<script lang="ts">
export let width: number = null;
export let onlyDesktop = false;
export let eyebrow = true;
</script>

<div
style={width ? `--p-col-width:${width?.toString()}` : ''}
class:is-only-desktop={onlyDesktop}
class="table-thead-col"
role="columnheader">
<span class="eyebrow-heading-3">
<span class={eyebrow ? 'eyebrow-heading-3' : 'body-text-2'}>
<slot />
</span>
</div>
Original file line number Diff line number Diff line change
Expand Up @@ -52,9 +52,9 @@
{#if data.documents.total}
<TableScroll isSticky>
<TableHeader>
<TableCellHead>Document ID</TableCellHead>
<TableCellHead eyebrow={false}>Document ID</TableCellHead>
{#each columns as column}
<TableCellHead>{column.title}</TableCellHead>
<TableCellHead eyebrow={false}>{column.title}</TableCellHead>
{/each}
</TableHeader>
<TableBody>
Expand Down

1 comment on commit 6fca232

@vercel
Copy link

@vercel vercel bot commented on 6fca232 Jan 27, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.