Mailing List Archive

Re: [8540] Make the Add keywords button (on the story and media profiles)
I think it was on the <tr> so that the word "Keywords:" lined up with
the first keyword. If it's not lined up anymore, you might want to add
it to the <th> too.


On Apr 1, 2009, at 6:49 PM, brewt@bricolage.cc wrote:

> Revision
> 8540
> Author
> brewt
> Date
> 2009-04-01 15:49:40 -0700 (Wed, 01 Apr 2009)
> Log Message
>
> Make the Add keywords button (on the story and media profiles) not
> so huge in IE. IE was making the buttons inherit the line-height
> from tr element, so I just moved the style into the keyword ul list
> instead.
> Modified Paths
>
> bricolage/trunk/comp/widgets/media_prof/edit_meta.html
> bricolage/trunk/comp/widgets/profile/fast_add.mc
> bricolage/trunk/comp/widgets/story_prof/edit_meta.html
> bricolage/trunk/lib/Bric/Changes.pod
> Diff
>
> Modified: bricolage/trunk/comp/widgets/media_prof/edit_meta.html
> (8539 => 8540)
>
> --- bricolage/trunk/comp/widgets/media_prof/edit_meta.html
> 2009-04-01 20:17:32 UTC (rev 8539)
> +++ bricolage/trunk/comp/widgets/media_prof/edit_meta.html
> 2009-04-01 22:49:40 UTC (rev 8540)
> @@ -208,7 +208,7 @@
> </tr>
> % } # if ENABLE_CATEGORY_BROWSER
>
> -<tr class="<% $rowColor++ % 2 == 0 ? "even" : "odd" %>" style="line-
> height: 2em">
> +<tr class="<% $rowColor++ % 2 == 0 ? "even" : "odd" %>">
> <th><% $lang->maketext('Keywords') %>:</th>
> <td colspan="2">
> <& '/widgets/profile/fast_add.mc',
> Modified: bricolage/trunk/comp/widgets/profile/fast_add.mc (8539 =>
> 8540)
>
> --- bricolage/trunk/comp/widgets/profile/fast_add.mc 2009-04-01
> 20:17:32 UTC (rev 8539)
> +++ bricolage/trunk/comp/widgets/profile/fast_add.mc 2009-04-01
> 22:49:40 UTC (rev 8540)
> @@ -1,4 +1,4 @@
> -<ul id="fast-add-<% $type %>">
> +<ul id="fast-add-<% $type %>" style="line-height: 2em">
> % foreach my $obj (@$objects) {
> <li class="<% $type %>">
> <& '/widgets/profile/hidden.mc',
> Modified: bricolage/trunk/comp/widgets/story_prof/edit_meta.html
> (8539 => 8540)
>
> --- bricolage/trunk/comp/widgets/story_prof/edit_meta.html
> 2009-04-01 20:17:32 UTC (rev 8539)
> +++ bricolage/trunk/comp/widgets/story_prof/edit_meta.html
> 2009-04-01 22:49:40 UTC (rev 8540)
> @@ -234,7 +234,7 @@
> </tr>
> % }
>
> -<tr class="<% $rowColor++ % 2 == 0 ? "even" : "odd" %>" style="line-
> height: 2em">
> +<tr class="<% $rowColor++ % 2 == 0 ? "even" : "odd" %>">
> <th><% $lang->maketext('Keywords') %>:</th>
> <td colspan="2">
> <& '/widgets/profile/fast_add.mc',
> Modified: bricolage/trunk/lib/Bric/Changes.pod (8539 => 8540)
>
> --- bricolage/trunk/lib/Bric/Changes.pod 2009-04-01 20:17:32 UTC
> (rev 8539)
> +++ bricolage/trunk/lib/Bric/Changes.pod 2009-04-01 22:49:40 UTC
> (rev 8540)
> @@ -425,6 +425,11 @@
>
> Xinha works again. Reported by John Durkin and Matt Rolf (Bug
> #1429). [David]
>
> +=item *
> +
> +Make the Add keywords button (on the story and media profiles) not
> so huge in
> +IE. [Adrian Yee]
> +
> =back
>
> =head1 VERSION 1.11.1 (2008-10-03)
Re: [8540] Make the Add keywords button (on the story and media profiles) [ In reply to ]
Marshall Roch wrote:
> I think it was on the <tr> so that the word "Keywords:" lined up with
> the first keyword. If it's not lined up anymore, you might want to add
> it to the <th> too.

Yup, you're right. Fixed.