Customize Address Book toolbars

RESOLVED FIXED in seamonkey2.1a3

Status

SeaMonkey
MailNews: Address Book & Contacts
--
enhancement
RESOLVED FIXED
7 years ago
4 years ago

People

(Reporter: Ian Neal, Assigned: Ian Neal)

Tracking

(Blocks: 1 bug)

Trunk
seamonkey2.1a3
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 3 obsolete attachments)

(Assignee)

Description

7 years ago
Other parts of SM have customizable toolbars so AB should too.
(Assignee)

Updated

7 years ago
Assignee: nobody → iann_bugzilla
Status: NEW → ASSIGNED
(Assignee)

Comment 1

7 years ago
Created attachment 455194 [details] [diff] [review]
WIP patch

This is very much WIP
Still need to move default buttons out of the palette, do CSS work and find out why the customization doesn't work
(Assignee)

Comment 2

7 years ago
Created attachment 455282 [details] [diff] [review]
Customizable toolbar patch v0.1

This patch:
* Makes toolbar in the address book customizable

I've tried having the default icons outside of the toolbarpalette but you just end up with icons you cannot move from the toolbar.
Attachment #455194 - Attachment is obsolete: true
Attachment #455282 - Flags: review?(philip.chee)
(Assignee)

Comment 3

7 years ago
(In reply to comment #2)
> Created an attachment (id=455282) [details]
> Customizable toolbar patch v0.1
> 
> This patch:
> * Makes toolbar in the address book customizable
> 
> I've tried having the default icons outside of the toolbarpalette but you just
> end up with icons you cannot move from the toolbar.

Oh I also changed the id for delete to make it unique.

Comment 4

7 years ago
Comment on attachment 455282 [details] [diff] [review]
Customizable toolbar patch v0.1

Bug 575956 - Customize Address Book toolbars

> I've tried having the default icons outside of the toolbarpalette but you just
> end up with icons you cannot move from the toolbar.

Strange. That's what Minefield does. I wonder what we/they are doing differently.

> Oh I also changed the id for delete to make it unique.
Unique in what sense? FYI Thunderbird uses "button-abdelete"

> +  <toolbar type="menubar"
> +           id="ab-toolbar-menubar2"

Thunderbird uses: "addrbook-toolbar-menubar2" Too verbose, but I suppose Mnyromyr will insist on following Thunderbird slavishly.

> +      <menubar id="ab-menubar">

Thunderbird uses: "mail-menubar" Too generic I fear. Perhaps Thunderbird should change instead of us?

> +  <toolbar class="chromeclass-toolbar toolbar-primary"
> +           id="abToolbar"

Thunderbird->"ab-bar2". Sigh.

> +    <toolbaritem id="searchBox"

Thunderbird uses "search-container". *We* use "search-container" in mailnews (IIRC it was your patch too). Note must change the .js as well. However it is unclear why Thunderbird increased the timeout to 800ms though.

> +    <toolbaritem id="throbber-box" align="center">

In small icon mode (e.g. move it to the menubar) the background should be removed e.g. { background: none; }. Bonus do it for the main mailnews window as well.

Unfortunately you will have to remove the list-style-image from .toolbarbutton-1

> .toolbarbutton-1 {
>   list-style-image: url("chrome://messenger/skin/addressbook/icons/addressbookicons.png");
> }

And apply it to each individual button. See the Thunderbird /skin/addressbook.css for example.

> -#button-delete {
> +#button-delete-item {

   list-style-image: url("chrome://messenger/skin/addressbook/icons/addressbookicons.png");

>    -moz-image-region: rect(0 29px 29px 0);
>  }

Please file bugs in classic and in modern for adding small toolbar icons to the Addressbook. For classic a suitable image resizing tool on the existing large icons would do. Modern would need completely new images.

Updated

7 years ago
Attachment #455282 - Flags: review?(philip.chee) → review-
(Assignee)

Updated

7 years ago
Blocks: 576402
(Assignee)

Updated

7 years ago
Blocks: 576403
(Assignee)

Comment 5

7 years ago
Created attachment 455595 [details] [diff] [review]
Customizable toolbar patch v0.1a

Changes since v0.1:
* Renamed id "ab-toolbar-menubar2" to "addrbook-toolbar-menubar2"
* Renamed id "abToolbar" to "ab-bar2"
* throbber-box for small icon toolbars now has no background on modern
* Removed the list-style-image from .toolbarbutton-1
* Added list-style-image, where relevant, to the individual buttons

Tried having one button still left in the palette and that was the only button that could be moved, all the rest were locked in place.

Left "ab-menubar" and "searchBox" ids alone so that they are unique in SM.
Attachment #455282 - Attachment is obsolete: true
Attachment #455595 - Flags: review?(philip.chee)

Comment 6

7 years ago
Comment on attachment 455595 [details] [diff] [review]
Customizable toolbar patch v0.1a

Looks good.
Attachment #455595 - Flags: review?(philip.chee) → review+

Comment 7

7 years ago
>> I've tried having the default icons outside of the toolbarpalette but you just
>> end up with icons you cannot move from the toolbar.
> 
> Strange. That's what Minefield does. I wonder what we/they are doing
> differently.

A closer reading of Bug 354048 indicates that we need to add |removable="true"| to those items that are already on a toolbar. Items in the palette automatically gain this attribute when moved to a toolbar.
(Assignee)

Comment 8

7 years ago
Created attachment 456965 [details] [diff] [review]
Customizable toolbar with removable patch v0.1b [Checkin: Comment 13]

Changes since v0.1a:
* Make use of |removable="true"|
Attachment #455595 - Attachment is obsolete: true
Attachment #456965 - Flags: superreview?(bugzilla)
Attachment #456965 - Flags: review?(philip.chee)

Comment 9

7 years ago
Comment on attachment 456965 [details] [diff] [review]
Customizable toolbar with removable patch v0.1b [Checkin: Comment 13]

Looks good. r=me
Attachment #456965 - Flags: review?(philip.chee) → review+
(Assignee)

Comment 10

7 years ago
(In reply to comment #9)
> Comment on attachment 456965 [details] [diff] [review]
> Customizable toolbar with removable patch v0.1b
> 
> Looks good. r=me

Just as a note I have one additional local change of <popup> to <menupopup>
(Assignee)

Updated

7 years ago
Attachment #456965 - Flags: superreview?(bugzilla) → superreview?(neil)
(In reply to comment #4)
> > +  <toolbar class="chromeclass-toolbar toolbar-primary"
> > +           id="abToolbar"
> Thunderbird->"ab-bar2". Sigh.
[Double sigh. Who was responsible for that id?]
Comment on attachment 456965 [details] [diff] [review]
Customizable toolbar with removable patch v0.1b [Checkin: Comment 13]

>+toolbar:not([iconsize="small"]) #throbber-box {
We only want the override to apply when the toolbar has a line, which is the specific case where the throbber is in a default primary non-navigator toolbar. So you either need to use .toolbar-primary:not([labelalign="end"]):not(etc.) or you need to override the background to transparent in each of the cases where toolbar.css overrides the line plus either way you also need to override the background to transparent in navigator.css which turns the line off anyway.
Attachment #456965 - Flags: superreview?(neil) → superreview+
(Assignee)

Comment 13

7 years ago
Comment on attachment 456965 [details] [diff] [review]
Customizable toolbar with removable patch v0.1b [Checkin: Comment 13]

http://hg.mozilla.org/comm-central/rev/6368a42a985a
Pushed with popup->menupopup and css rule change.
Attachment #456965 - Attachment description: Customizable toolbar with removable patch v0.1b → Customizable toolbar with removable patch v0.1b [Checkin: Comment 13]
(Assignee)

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED

Updated

7 years ago
Target Milestone: --- → seamonkey2.1a3

Updated

4 years ago
Depends on: 882178
You need to log in before you can comment on or make changes to this bug.