Skip to content
Snippets Groups Projects
Unverified Commit 9dfdd9ec authored by Saurabh Kumar's avatar Saurabh Kumar Committed by GitHub
Browse files

Fix alignment of expand icon + make it accessible (#3917)

Fixes #3889
parent bdac9950
No related branches found
No related tags found
1 merge request!89Upgrade to 5.10.0
...@@ -4,11 +4,12 @@ ...@@ -4,11 +4,12 @@
let submission_arrow = document.createElement("span"); let submission_arrow = document.createElement("span");
submission_arrow.classList.add("arrow"); submission_arrow.classList.add("arrow");
let submission_toggle = document.createElement("span"); let submission_toggle = document.createElement("button");
submission_toggle.classList.add( submission_toggle.classList.add(
"all-submissions-table__toggle", "all-submissions-table__toggle",
"js-toggle-submission" "js-toggle-submission"
); );
submission_toggle.setAttribute("title", "Show submission details");
submission_toggle.appendChild(submission_arrow); submission_toggle.appendChild(submission_arrow);
// Add the toggle arrow before the submission titles. // Add the toggle arrow before the submission titles.
......
...@@ -161,7 +161,6 @@ ...@@ -161,7 +161,6 @@
@include triangle(right, $color--primary, 6px); @include triangle(right, $color--primary, 6px);
position: relative; position: relative;
display: inline-block; display: inline-block;
margin-inline-end: 10px;
transform: rotate(0); transform: rotate(0);
transition: transform $transition; transition: transform $transition;
...@@ -245,7 +244,14 @@ ...@@ -245,7 +244,14 @@
} }
&__toggle { &__toggle {
padding: 5px 0 5px 5px; padding: 0.5em;
margin-top: -0.5em;
cursor: pointer;
vertical-align: top;
&:hover {
opacity: 0.7;
}
} }
&__more { &__more {
......
...@@ -116,7 +116,7 @@ ...@@ -116,7 +116,7 @@
} }
&.title { &.title {
padding-inline-start: 20px; padding-inline-start: 0.5em;
} }
} }
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment