Closed (fixed)
Project:
Localization server
Version:
6.x-2.x-dev
Component:
Documentation
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
27 Feb 2010 at 11:35 UTC
Updated:
24 Mar 2010 at 08:10 UTC
Jump to comment: Most recent file
Comments
Comment #1
gábor hojtsyAgreed. This is a bit more trickier compared to the "Edit a copy" button, because these are checkboxes converted to (hopefully) more meaningful action icons. So we do not have control in the Form API to directly put a title on the label element. We could either trick it with some overriden theming or within editor.js. The trick is that the decline and undecline icons are representations of the state of one single checkbox, so we'd need to change the lable title as the state of the checkbox changes.
This is our current markup:
Comment #2
gábor hojtsyOk, took suggestions from http://localize.drupal.org/node/993#comment-4008, quoted the relevant items:
Was munching on this a bit, and came up with this:
- renamed "Edit a copy" to "Translate" when belonging to the source string (its evident you edit a copy)
- kept "Edit a copy" as-is when copying and editing another suggestion, IMHO otherwise its not clear you are not editing that text at all, but add another one
- Added hover tips to the "decline" and "undecline" icons; since these are added dynamically as discussed above, the tips are added and modified via JS; namely "Decline" and "Undo decline"
- Modified "Show/Hide occurrences" to "Show/Hide affected projects" and added hover tip: "List of projects and releases where this text is used."
Hope this makes it even clearer.
I was looking hard at the attribution text and do not know how could we make it shorter, so did not touch that.
I'm not adding a screenshot since this is all just text changes.
Comment #3
gábor hojtsyThinking this more, I've renamed the "affected projects" to "related projects". Since this shows up with the source string, not the translation, and translation has an effect, not the source string. The tooltip is kept as useful as in the previous patch (if it is :).
Committing for deployment on staging, since I've already had feedback on these issues independently, so better to let people test an updated version.
Comment #4
gábor hojtsyOh, this is the patch that's going to be committed.
Comment #5
gábor hojtsyLooks like the show/hide also needs a "Loading..." state, since loading that data takes some time, and the user would probably not be aware. So adding replacement text to "Loading..." while loads, and back to Hide related projects when shown. Committing this quick update.