Conversation
| [matTooltip]="element.reference_id" | ||
| matTooltipPosition="above" | ||
| class="cursor-pointer font-mono text-xs px-2 py-0.5 rounded-4 min-w-[150px]" | ||
| class="cursor-default" |
There was a problem hiding this comment.
This change removes several styling classes (font-mono text-xs px-2 py-0.5 rounded-4 min-w-[150px]) that affect the appearance of the reference ID display. Was this intentional? If the goal was just to change the cursor style, you might want to keep the other styling classes to maintain the visual design.
| <!-- ── Snippet Tab ── --> | ||
| @if (selectedSectionIndex === 2) { | ||
| <div class="flex flex-col gap-4 max-w-2xl"> | ||
| <div class="flex flex-col gap-4"> |
There was a problem hiding this comment.
Removing the max-width constraint from the snippet container could make code snippets stretch too wide on larger screens, reducing readability. Consider keeping a reasonable maximum width.
| <div class="card-with-header--header justify-between"> | ||
| <div class="flex items-center gap-2"> | ||
| <mat-icon class="!text-base !w-4 !h-4 secondary-text-color opacity-70">html</mat-icon> | ||
| <mat-icon>html</mat-icon> |
There was a problem hiding this comment.
The HTML icon is missing the same sizing classes that are applied to the code icon above it. This will cause inconsistent icon sizing in the UI.
| <mat-icon>html</mat-icon> | |
| <mat-icon class="!text-base !w-4 mt-1">html</mat-icon> |
No description provided.