Comments

cgalli’s picture

Status: Active » Needs review
miro_dietiker’s picture

Works. Unsure about how much details we should provide here. Possibly fine for a start.

I just realised that the Cart translate tab shows "Translation" only as title.
Do we lack a real title? The other tabs all have some custom title.

cgalli’s picture

StatusFileSize
new2.25 KB

Trial. Added help text for each source page.
Looks a bit crowded.

cgalli’s picture

@miro
Translation Cart it should be!

berdir’s picture

+++ b/ui/tmgmt_ui.module
@@ -1180,6 +1215,13 @@ function tmgmt_ui_help($path, $arg) {
+
+    // Find the type of source.
+    $source = arg(3) != NULL ? arg(3) : 'content';

This is unfortunately not that easy. The default is dynamic, based on what you have available.

I'd just leave this out and just say items or sources, something like locale_default items isn't useful either.

berdir’s picture

Status: Needs review » Needs work
cgalli’s picture

Status: Needs work » Needs review
StatusFileSize
new3.5 KB

Removed source type
Adusted css for overlay display

Status: Needs review » Needs work

The last submitted patch, 7: help_text_for_cart_7.patch, failed testing.

cgalli’s picture

Status: Needs work » Needs review
StatusFileSize
new2.97 KB

Reversed some style changes...

cgalli’s picture

StatusFileSize
new574 bytes

Alternatively to putting help text on every source page we could add a tool tip to all buttons.

The patch adds a 'title' attribute to the 'Add to cart' button

Status: Needs review » Needs work

The last submitted patch, 10: help_text_for_cart_10.patch, failed testing.

blueminds’s picture

Status: Needs work » Needs review
StatusFileSize
new2.57 KB

Rerolled #9, did minor cleanup.

blueminds’s picture

StatusFileSize
new2.91 KB

and with the title attribute for the add to cart button

blueminds’s picture

StatusFileSize
new1.65 KB

and removing source overview help text

The last submitted patch, 12: help_text_for_cart_11.patch, failed testing.

berdir’s picture

Status: Needs review » Fixed

Committed and pushed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.