Question

I need to show multiple buttons, but each one must have a different background than other buttons, I have been working on it, but I only got to display multiple buttons but with the same background.
Here is the XAML:

<Window x:Class="apple.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        Title="MainWindow" Height="370" Width="525">
    <Grid>
        <Image Source="C:\Users\Public\Pictures\Sample Pictures\Koala.jpg" Stretch="Fill"/>
        <DockPanel Name="dock">
            <UniformGrid Name="gridx" DockPanel.Dock="Top" Rows="3" Columns="3" Height="334"> 
            </UniformGrid>
        </DockPanel>
    </Grid>
</Window>

Also, here is the C# code:

namespace apple
{
    public partial class MainWindow : Window
    {
        public MainWindow()
        {
            InitializeComponent();
            masterGUI();
        }

    public void masterGUI()
    {
        ImageBrush ib = new ImageBrush();
        IconImage[] ico = null;
        Bitmap[] img = null;
        string[] list = null;
        string[] link = Directory.GetFiles(@"C:\ProgramData\Microsoft\Windows\Start Menu\Programs", "*.lnk", SearchOption.AllDirectories);
        list = new string[link.Length];
        ico = new Icon[link.Length];
        img = new Bitmap[link.Length];
        for (int n = 0; n < link.Length; n++)
        {
            System.Windows.Controls.Button newBtn = new Button();
            list[n] = System.IO.Path.GetFileNameWithoutExtension(link[n]);
            FileToImageIconConverter some = new FileToImageIconConverter(link[n]);
            ImageSource imgSource = some.Icon;
            ib.ImageSource = imgSource;
            newBtn.Background = ib;
            newBtn.Content = list[n];
            gridx.Children.Add(newBtn);
        }  
    }
}

}

Any idea? thank you.

Answer 1

The ImageBrush needs to be created in the for-loop individually for each item. Otherwise you will end up with the same background for every item.

Also you are approaching this the “wrong” way, in WPF you should use data binding and data templating for this sort of thing instead of imperative looping.