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 411 forks source link

Toolbar improvements in LabyrinthJS.activity #1073

Closed S-kus closed 1 year ago

S-kus commented 2 years ago

  1. Currently sub-toolbar is overlapping with activity-palette, as shown:

    laby1

    It should be like this: 2laby

  2. The button title of Idea text and the background color is Stroke Color and Fill Color respectively, As shown:

    Screenshot from 2022-05-08 16-38-55

    It can be set to Text Color and Background Color for more specificity.

S-kus commented 2 years ago

@llaske I raised the pull request for 1st issue. Could you please review it and also suggest is 2nd issue needed to be solved or not? If it's required, should I just update the title message in the locale.ini file, or is anything else required? I will commit for this change in the same pull request.

llaske commented 2 years ago

You're right for 1. I don't think the text for 2 is confusing.

S-kus commented 2 years ago

Thanks, @llaske, I have fixed the 1st issue. Could you please review this pull request?

llaske commented 2 years ago

Fixed in https://github.com/llaske/sugarizer/pull/1074