Opened 13 years ago
Closed 11 years 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 13 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)
comment:2 Changed 13 years ago by cjl
- Component changed from Measure to Paint
- Owner changed from walter to godiard
comment:3 Changed 13 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 13 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: ↓ 7 Changed 13 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 13 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 13 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 11 years ago by godiard
- Resolution set to fixed
- Status changed from new to closed
Correcting reporting error, this is an issue with Paint, not Measure.