Completed
Last Updated: 28 Nov 2024 15:16 by Joshua
Release 7.0.0
David
Created on: 02 Feb 2023 11:33
Category: Drawer
Type: Bug Report
11
When using the Template, the component must not trap any HTML events, as the keyboard navigation of the nested components does not work

Cases:

  • I am using the Template to control the rendering of the Drawer. When I place an <a> tag and focus it, I cannot use the Enter key to navigate to the desired link. The component traps the onkeypress event, which it should not.
  • When I place a TreeView inside the Drawer Template and focus it, the keyboard navigation does not work because the Drawer traps the HTML events like onkeydown.

<Admin>

Workaround for the first case:

<script suppress-error="BL9992">
    window.navigateToHref = (ev) => {
        if (ev.key === "Enter") {
            location.href = ev.target.href
        }
    };
</script>

@* This example shows how to create header and footer for the Drawer and select an item manually. *@
<TelerikDrawer @bind-Expanded="@DrawerExpanded"
               Data="@Data"
               MiniMode="true"
               Mode="@DrawerMode.Push"
               @bind-SelectedItem="@SelectedItem"
               @ref="@DrawerRef">
    <Template>
        @* the header *@
        <div>
            <TelerikButton OnClick="@(() => DrawerRef.ToggleAsync())" Icon="@FontIcon.Menu" />
            @if (DrawerExpanded)
            {
                <div class="text-info" style="border-bottom:solid; font-weight: bold; margin-bottom: 3em; white-space:nowrap">
                    <a href="https://google.com" onkeydown="navigateToHref(event)">
                        My Custom Navigation to Google
                    </a>
                </div>
            }
            else
            {
                <div class="text-info" style="border-bottom:solid; font-weight: bold;">
                    Nav
                </div>
            }
        </div>

        @* custom items rendering and item selection *@

        <div class="k-drawer-items">
            <ul>
                @if (SelectedItem != null && DrawerExpanded)
                {
                    <li class="k-drawer-item" style="white-space:nowrap">
                        <div>
                            <p><strong>@SelectedItem.Text</strong></p>
                            <p>@SelectedItem.Description</p>
                        </div>
                    </li>
                }

                @foreach (var item in Data)
                {
                    @* Use onclick to handle manual item selection *@
                    <li @onclick="@(() => SelectedItem = item)"
                    class="k-drawer-item @GetSelectedItemClass(item)" style="white-space:nowrap">
                        <TelerikFontIcon Icon="@item.Icon"></TelerikFontIcon>
                        @if (DrawerExpanded)
                        {
                            <div>
                                <div>@item.Text</div>
                            </div>
                        }
                    </li>
                }
            </ul>
        </div>

        @* the footer *@
        @if (DrawerExpanded)
        {
            <div style="text-align: center; margin-top: 3em; padding-top: 2em; border-top: 2px solid black; white-space:nowrap">
                <img src="user-avatar.png" alt="my avatar" style="border-radius: 50%; width: 50px; height: 50px;" />
                <br /><br />
                <TelerikButton Icon="@FontIcon.Logout" ThemeColor="primary">Log Out</TelerikButton>
            </div>
        }
    </Template>
    <DrawerContent>
        <div class="m-5">Content for @SelectedItem?.Text - @SelectedItem?.Description</div>
    </DrawerContent>
</TelerikDrawer>


@code {
    public TelerikDrawer<DrawerItem> DrawerRef { get; set; }
    public DrawerItem SelectedItem { get; set; }
    public bool DrawerExpanded { get; set; } = true;
    public IEnumerable<DrawerItem> Data { get; set; } = new List<DrawerItem>
    {
        new DrawerItem {Text = "Shopping Cart", Icon = FontIcon.Cart, Description = "Items in shopping cart"},
        new DrawerItem {Text = "Settings", Icon = FontIcon.Gear, Description = "My profile settings"},
        new DrawerItem {Text = "Notifications", Icon = FontIcon.ExclamationCircle, Description = "My profile notifications"},
        new DrawerItem {Text = "Calendar", Icon = FontIcon.Calendar, Description = "My events"},
    };

    public string GetSelectedItemClass(DrawerItem item)
    {
        if (SelectedItem == null) return string.Empty;
        return SelectedItem.Text.ToLowerInvariant().Equals(item.Text.ToLowerInvariant()) ? "text-info" : "";
    }

    public class DrawerItem
    {
        public string Text { get; set; }
        public FontIcon? Icon { get; set; }
        public string Description { get; set; }
    }
}

</Admin>

5 comments
Joshua
Posted on: 28 Nov 2024 15:16

Thanks Nadezhda, 

 

We have upgraded to the November 2024 release and resolved this bug in our solution which is much appreciated.

 

Many Thanks,

Joshua Burrowes

ADMIN
Nadezhda Tacheva
Posted on: 13 Aug 2024 08:55

Hi Joshua,

I discussed the item with the team and confirmed we can plug it in our release in November. We updated the status of the item.

Regards,
Nadezhda Tacheva
Progress Telerik

Do you have a stake in the designеr-developer collaboration process? If so, take our survey to share your perspective and become part of this global research. You’ll be among the first to know once the results are out.
-> Start The State of Designer-Developer Collaboration Survey 2024

Joshua
Posted on: 06 Aug 2024 18:53
Is there any timeframe of when a fix for this bug will be planned in?
Alexander
Posted on: 15 Jul 2024 11:38

From an accessibility perspective this is not good. I am under the impression that accessibility has recently been a major concern of "Telerik.UI.for.Blazor", therefore I am surprised to learn that this issue is known for over a year and nothing has been done.

Especially considering that in my opinion it would be fairly simple to solve: just don't register the "onkeydown" event handler in the "TelerikDrawer" if a "Template" is specified.

 

Code at the moment:

Code with fix (?):

Alexander
Posted on: 15 Jul 2024 11:34

We also came across this problem when trying to use a "TelerikButton" in the "Template". When the button has focus, it can be activated using the "Space" key, but it cannot be activated using the "Enter" key. Outside of the drawer template it is possible, however, to use the "Enter" key. Digging a bit into the source reveals that the "TelerikDrawer" registers an "onkeydown" event handler via JavaScript to intercept calls to "ArrowUp", "ArrowDown" as well as "Enter" to make the component keyboard navigable when using an "ItemTemplate" or no templates at all. The event handler calls "event.preventDefault", which leads to the problem described above when using a "Template". 


Here is a REPL showcasing the problem: https://blazorrepl.telerik.com/QIurbfEB089CF1aR11


Click on any of the two buttons, notice how the counter increases. As the button now has the focus, you can press spacebar and see the counter increasing again. If you press enter, however, the counter only increases for the button outside of the drawer "Template".