search

llaske / sugarizer

Sugarizer is a web implementation of the Sugar platform to run on any device or browser
https://sugarizer.org
Apache License 2.0
197 stars 412 forks source link

LabyrinthJS: Idea selection-indicator while linking ideas #1584

Closed 44yu5h closed 4 months ago

44yu5h commented 5 months ago

While linking ideas, no selection-indicator of selected idea was visible thereby creating confusion. Added a selection-indicator to the currently selected idea and auto-deselect after making a link.

VishnuVardhanBR commented 5 months ago

Please avoid sending a PR before creating an issue which should also be reviewed by @llaske.

44yu5h commented 5 months ago

@VishnuVardhanBR It is a minor UI/UX impovement. I don't think this needs an issue to be created. We can discuss here if there's anything to. But I'm new to this community, if that's what people follow here then I shall do so :)

llaske commented 5 months ago

Keyboard shortcuts are useless in Sugarizer because it's too complex for children and because Sugarizer is mostly use on tablets. As @VishnuVardhanBR suggest, please open an issue before sending a PR.

44yu5h commented 5 months ago

@llaske Removed shortcuts. Thanks.

Issue #1585

llaske commented 4 months ago

Nice. Thanks.