Opened 4 years ago

Closed 18 months ago

#2675 closed defect (fixed)

Paint "View Toolbar" not implemented

Reported by: cjl Owned by: RafaelOrtiz
Priority: Unspecified by Maintainer Milestone: Unspecified
Component: localization Version: Unspecified
Severity: Unspecified Keywords:
Cc: godiard Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description (last modified by cjl)

Testing Paint-31 in 11.2.0 os12 on XO-1.

In the git repo, I see code for a View Toolbar from toolbox.py:1471 to the end of the file, but this feature is not visible when running the activity (ZOOM + / ZOOM -). This section of code includes a lot of strings that are in the POT file but not visible when running the activity.

If implemented (or if I just didn't find it), it would be desirable for ZOOM + / ZOOM - strings to be standardized to "Zoom in" / "Zoom out" like it is in most other Activities for Sugar UI consistency.

Change History (8)

comment:1 Changed 4 years ago by cjl

  • Description modified (diff)
  • Summary changed from Measure "View Toolbar" not discoverable (or not implemented) to Paint "View Toolbar" not discoverable (or not implemented)

Correcting reporting error, this is an issue with Paint, not Measure.

comment:2 Changed 4 years ago by cjl

  • Component changed from Measure to Paint
  • Owner changed from walter to godiard

comment:3 Changed 4 years ago by godiard

  • Summary changed from Paint "View Toolbar" not discoverable (or not implemented) to Paint "View Toolbar" not implemented

Zoom is not implemented in Paint.

There are other tickets opened about this, like https://dev.laptop.org/ticket/3688

comment:4 Changed 4 years ago by cjl

I'm okay if Zoom is not implemented, but we should drop the last 10 strings from the POT file if that is going to remain the status quo.

comment:5 follow-up: Changed 4 years ago by godiard

Texts removed in version 32

Commit http://git.sugarlabs.org/paint/mainline/commit/31860cd480790e97353a0faef0cf29303ca9b7c5

We need see why the strings are present in pootle right now. (I think pootleupdater add strings but not remove the old ones)

comment:6 Changed 4 years ago by cjl

  • Cc godiard added
  • Component changed from Paint to localization
  • Owner changed from godiard to RafaelOrtiz

Changing this to component = localization and adding godiard as a CC: as owner will change to RafaelOrtiz

comment:7 in reply to: ↑ 5 Changed 4 years ago by RafaelOrtiz

Replying to godiard:

Texts removed in version 32

Commit http://git.sugarlabs.org/paint/mainline/commit/31860cd480790e97353a0faef0cf29303ca9b7c5

We need see why the strings are present in pootle right now. (I think pootleupdater add strings but not remove the old ones)

That's a good guess, not sure why people haven't seen this before.
afaik, potupdater is only updating the strings, we have to check that.

comment:8 Changed 18 months ago by godiard

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.