Skip to content

Commit

Permalink
Improve focus styles on Doc::TileMedium (#663)
Browse files Browse the repository at this point in the history
* Improve focus styles on Doc::TileMedium

* Empty-Commit
  • Loading branch information
jeffdaley authored Apr 1, 2024
1 parent b3384ef commit 8aaf601
Show file tree
Hide file tree
Showing 8 changed files with 9 additions and 5 deletions.
2 changes: 1 addition & 1 deletion web/app/components/dashboard/index.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

<div class="flex w-full flex-col-reverse gap-5 lg:flex-row lg:gap-0">
{{! Main }}
<div class="min-h-[500px] w-full overflow-hidden pb-20">
<div class="min-h-[500px] w-full pb-20">
<div class="main pr-4">
{{#if @docsAwaitingReview}}
<Dashboard::DocsAwaitingReview @docs={{@docsAwaitingReview}} />
Expand Down
3 changes: 2 additions & 1 deletion web/app/components/doc/tile-medium.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -35,11 +35,12 @@
@route="authenticated.document"
@query={{hash draft=this.docIsDraft}}
@model={{this.docID}}
class="group/primary-link"
>
{{! Primary click area }}
<div
aria-hidden="true"
class="absolute -top-px left-0 -bottom-px w-full mix-blend-multiply group-focus-within:bg-color-page-faint group-focus-within:outline group-focus-within:outline-2 group-focus-within:outline-color-border-strong group-hover:bg-color-page-faint"
class="absolute -top-px left-0 -bottom-px w-full mix-blend-multiply group-hover:bg-color-page-faint group-focus/primary-link:outline group-focus/primary-link:outline-2 group-focus/primary-link:outline-color-border-strong"
></div>

<h3 class="relative">
Expand Down
1 change: 1 addition & 0 deletions web/app/components/document/sidebar.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -351,6 +351,7 @@
</LinkTo>

<OverflowMenu
@renderOut={{true}}
@items={{hash
remove=(hash
icon="trash"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
</ExternalLink>
{{/if}}
{{#unless @editingIsDisabled}}
<OverflowMenu @items={{this.overflowMenuItems}} />
<OverflowMenu @items={{this.overflowMenuItems}} @renderOut={{true}} />
{{/unless}}
</div>
</li>
Expand Down
2 changes: 1 addition & 1 deletion web/app/components/overflow-menu.hbs
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<X::DropdownList
@items={{@items}}
@renderOut={{true}}
@renderOut={{@renderOut}}
@placement="bottom-end"
data-test-overflow-menu
...attributes
Expand Down
1 change: 1 addition & 0 deletions web/app/components/overflow-menu.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ interface OverflowMenuComponentSignature {
items: Record<string, OverflowItem>;
offset?: OffsetOptions;
isShown?: boolean;
renderOut?: boolean;
};
}

Expand Down
2 changes: 1 addition & 1 deletion web/app/components/project/index.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -189,7 +189,7 @@
We add a page background color to overlay any
removed documents that are still animating out.
}}
<div class="relative z-10 bg-color-page-primary pt-8">
<div class="relative z-10 mt-1 bg-color-page-primary pt-7">
{{! External links }}
<Project::ResourceList
data-test-external-link-list
Expand Down
1 change: 1 addition & 0 deletions web/app/components/project/jira-widget.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,7 @@
}}
@isShown={{true}}
@offset={{hash mainAxis=1 crossAxis=-3}}
@renderOut={{true}}
/>
{{/unless}}
{{else if @isLoading}}
Expand Down

0 comments on commit 8aaf601

Please sign in to comment.