Minor UX tweaks for icons + color sections

[Niels Laute (niels9001)]

This PR introduces the following changes:

• Tweaks the UX of the Icon panel for better readability.
• Using an ItemsView without buttons, for better accessibility. Using a WrapLayout (from the WCT) so labels are not cut off.
• Selecting the first item upon loading, or after searching.
• Collapse the sidepanel until the grid icons is loaded: current behavior shows empty content on the right for 2 seconds which does not look great.

• Spacing tweaks for the color sections so they are clustered together a bit better
• Using a 0.8 opacity for the subtitle to be a bit less prominent

[Zakaria Tahri (Zakariathr22)]

Hi Niels Laute (@niels9001),

I believe there's a small issue. The actual tags for the Lock icon are:

{
  "Code": "E72E",
  "Name": "Lock",
  "Tags": [
    "private",
    "security",
    "access",
    "protected",
    "safety",
    "permission",
    "access",
    "protection"
  ]
}

However, the displayed tags don't match. It seems the app is showing the initial tags from the previously selected icon and then combining them with the remaining tags from the currently selected icon (you can compare the main branch with the new branch to observe the difference).

[Zakaria Tahri (Zakariathr22)]

Niels Laute (@niels9001)

It seems the issue arises from the WrapLayout not correctly tracking changes in the items. This is what result in incorrect tags updates when the selected icon changes.

I suggest modifying the IconsItemsView_SelectionChanged method as follows:

private void IconsItemsView_SelectionChanged(ItemsView sender, ItemsViewSelectionChangedEventArgs args)
{
    if (IconsItemsView.ItemsSource is IList<IconData> currentItems)
    {
        if (IconsItemsView.CurrentItemIndex != -1 && IconsItemsView.CurrentItemIndex < currentItems.Count)
        {
            SelectedItem = currentItems[IconsItemsView.CurrentItemIndex];
        }
    }

    if (TagsItemsView != null)
    {
        TagsItemsView.Layout = new WrapLayout { VerticalSpacing = 4, HorizontalSpacing = 4 };
    }
}

This ensures that the layout for TagsItemsView is refreshed whenever the selection changes. By reassigning a new instance of WrapLayout with the same spacing values (VerticalSpacing = 4, HorizontalSpacing = 4), we effectively reset the layout. This forces the ItemsRepeater to re-measure and re-arrange the items, ensuring it reflects the current data accurately.

[Niels Laute (niels9001)]

Niels Laute (@niels9001)

It seems the issue arises from the WrapLayout not correctly tracking changes in the items. This is what result in incorrect tags updates when the selected icon changes.

I suggest modifying the IconsItemsView_SelectionChanged method as follows:

private void IconsItemsView_SelectionChanged(ItemsView sender, ItemsViewSelectionChangedEventArgs args)
{
    if (IconsItemsView.ItemsSource is IList<IconData> currentItems)
    {
        if (IconsItemsView.CurrentItemIndex != -1 && IconsItemsView.CurrentItemIndex < currentItems.Count)
        {
            SelectedItem = currentItems[IconsItemsView.CurrentItemIndex];
        }
    }

    if (TagsItemsView != null)
    {
        TagsItemsView.Layout = new WrapLayout { VerticalSpacing = 4, HorizontalSpacing = 4 };
    }
}

This ensures that the layout for TagsItemsView is refreshed whenever the selection changes. By reassigning a new instance of WrapLayout with the same spacing values (VerticalSpacing = 4, HorizontalSpacing = 4), we effectively reset the layout. This forces the ItemsRepeater to re-measure and re-arrange the items, ensuring it reflects the current data accurately.

Thanks.. something to follow up on with the Toolkit! Your workaround seems to work, thanks for the suggestion 😄!

[Zakaria Tahri (Zakariathr22)]

IMHO, we should keep the top margin 12 to ensure equal margins at the top and bottom? (like in the following pic)

[Zakaria Tahri (Zakariathr22)]

My last thought in this PR is that the paragraphs are too close together, making it hard to tell whether each part is a separate idea or connected to the previous one.

Wouldn't adding some space between paragraphs (margins) improve readability and clarity?

[Karen (karkarl)]

/azp run

[Karen (karkarl)]

/azp run