575797 - widgets should be able to modify their width

Status: RESOLVED FIXED

(Add-on SDK Graveyard :: General, defect)

Description

[Dietrich Ayala (:dietrich)]

widgets should be able to explicitly set a static width, as well as specify that the width should flex to the size of it's contents.

Comment 1

[Dietrich Ayala (:dietrich)]

Attachment: v1

* support static width in widget creation
* support for updating width post-creation

TODO:
* tests
* docs

i haven't found a nice way to get the widgets to automatically resize to the size of their content. i'll probably file a different bug for that, so we can at least get static sizing into 0.6.

Comment 2

[Dietrich Ayala (:dietrich)]

Attachment: v2

* tests
* docs

Comment 3

[Drew Willcoxon :adw]

Comment on attachment 457997 [details] [diff] [review]
v2

>     // TODO move into a stylesheet
>     node.setAttribute("style", [
>-        "min-height: 24px; min-width: 24px; max-height: 24px; max-width: 24px;",
>+        "min-height: 24px; max-height: 24px;",
>         "overflow: hidden; margin: 5px; padding: 0px;",
>+        (widget.width ? "min-width: " + widget.width : "24") + "px;",

This string is getting a little long.  Would be good to get rid of it in favor of updating node.style.minWidth, etc.

>+    // test widget.width
>+    tests.push(function() testSingleWidget(widgets.Widget({
>+      label: "test widget.width",
>+      content: "test width",
>+      width: 200,
>+      onReady: function(e) {
>+        test.assertEqual(this.width, 200);
>+        widgets.remove(this);
>+        doneTest();
>+      }
>+    })));

Since width can be updated (as the example shows), could you expand this slightly to check that it works?

BTW, I didn't actually test this in my browser, but I'm sure you did. :)

r=adw

Comment 4

[Dietrich Ayala (:dietrich)]

Attachment: v3

made the changes, also updating the same style property consistently now. carrying over review.

Comment 5

[Dietrich Ayala (:dietrich)]

http://hg.mozilla.org/labs/jetpack-sdk/rev/21575da79567

Comment 6

[Myk Melez [:myk] [@mykmelez]]

The Add-on SDK is no longer a Mozilla Labs experiment and has become a big enough project to warrant its own Bugzilla product, so the "Add-on SDK" product has been created for it, and I am moving its bugs to that product.

To filter bugmail related to this change, filter on the word "looptid".