Skip to content

Initialise LED layout in constructor of LogitechPerKeyRGBDevice class #448

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
DarthAffe merged 3 commits into from
Mar 19, 2026
Merged

Initialise LED layout in constructor of LogitechPerKeyRGBDevice class #448

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
b0acc61
Merge pull request #430 from DarthAffe/Development
DarthAffe Jul 17, 2025
cf1cb4a
Fix - Updated constructor of LogitechPerKeyRGBDevice to initialise LE…
megabytesme Mar 18, 2026
66597cf
Apply suggestion from @DarthAffe (add all LEDs in a line with default…
megabytesme Mar 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 12 additions & 1 deletion RGB.NET.Devices.Logitech/PerKey/LogitechPerKeyRGBDevice.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,14 +24,25 @@ internal LogitechPerKeyRGBDevice(LogitechRGBDeviceInfo info, IUpdateQueue update
: base(info, updateQueue)
{
this._ledMapping = ledMapping;

InitializeLayout();
}

#endregion

#region Methods

private void InitializeLayout()
{
foreach ((LedId ledId, LogitechLedId mapping) in _ledMapping)
{
// TODO: Impove layout (curently places them in a grid)
AddLed(ledId, new Point(0, 0), new Size(10, 10));
}
Comment thread
megabytesme marked this conversation as resolved.
Outdated
}

/// <inheritdoc />
protected override object GetLedCustomData(LedId ledId) => _ledMapping.TryGetValue(ledId, out LogitechLedId logitechLedId) ? logitechLedId : -1;

#endregion
}
Loading