Skip to content

Commit

Permalink
fix: Do not disable cell focus in Grid header
Browse files Browse the repository at this point in the history
refactor: Use Java API instead of JS
  • Loading branch information
TatuJLund committed Dec 28, 2024
1 parent a0ae138 commit bd9f956
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,7 @@ public class VaadinCreateUI extends UI implements EventBusListener, HasI18N {

@Override
protected void init(VaadinRequest request) {
setOverlayContainerLabel("");
getPage().setTitle("Vaadin Create 23'");
if (!getAccessControl().isUserSignedIn()) {
MDC.clear();
Expand Down Expand Up @@ -244,12 +245,10 @@ public void detach() {
@Override
public void attach() {
super.attach();
// Workaround: https://github.com/vaadin/framework/issues/5772
getPage().getJavaScript().execute(
"""
setTimeout(() => {
document.getElementsByClassName('v-tooltip')[0].style.zIndex=30001;
document.getElementsByClassName('v-overlay-container')[0].removeAttribute('aria-label');
}, 200);
""");
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
// Experimental style for the grid with no visible cell focus.
// This is not part of the Valo theme, but could be added to it.
.no-cell-focus {
.v-grid-cell-focused:before {
td.v-grid-cell-focused:before {
border: unset;
}
}
Expand Down

0 comments on commit bd9f956

Please sign in to comment.