1054661 - Resizeble bookmark properties panel in Bookmarks Manager

Status: NEW

(SeaMonkey :: Bookmarks & History, enhancement)

Description

[Peter B. Shalimoff]

Attachment: moz.bmkmgr.resizable_bmk_props_panel.patch

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:28.0) Gecko/20100101 SeaMonkey/2.25 (Beta/Release)
Build ID: 20140318183706

Steps to reproduce:

Not a bug, but a RFE. I often use the Description field to save some extra info, e.g. how I found this page, or the author's name and link, or some important parts of the text I want to stay available in case the bookmarked page is gone, etc. In its current state (about only 2 lines tall), the Description field is very inconvenient to use if you have more than 2 lines of text. This patch makes the bookmark properties panel resizable so that you could make the Description field as tall as you want.

Comment 1

[Peter B. Shalimoff]

Attachment: bmk_prop_fewer.png

Default view.

Comment 2

[Peter B. Shalimoff]

Attachment: bmk_prop_more_def.png

Default expanded/full view.

Comment 3

[Peter B. Shalimoff]

Attachment: bmk_prop_more_resized.png

Resized view.

Comment 4

[Philip Chee]

Comment on attachment 8474120 [details] [diff] [review]
moz.bmkmgr.resizable_bmk_props_panel.patch

setting f? to Neil (should also be a ui-r? too)

Comment 5

[[e-mail]]

Comment on attachment 8474120 [details] [diff] [review]
moz.bmkmgr.resizable_bmk_props_panel.patch

>+#editBMPanel_descriptionField {
>+  min-height: 4em;
> }
It would probably be better to put rows="3" on the textbox as this will guarantee an exact fit.

>       <splitter id="detailsDeck-splitter"
>                 collapse="after"
>-                persist="state collapsed"
>-                disabled="true">
>+                persist="state collapsed">
We can't do this because the details toggle depends on it.

Comment 6

[Peter B. Shalimoff]

(In reply to [e-mail] from comment #5)
> Comment on attachment 8474120 [details] [diff] [review]
> moz.bmkmgr.resizable_bmk_props_panel.patch
> 
> >+#editBMPanel_descriptionField {
> >+  min-height: 4em;
> > }
> It would probably be better to put rows="3" on the textbox as this will
> guarantee an exact fit.

That's exactly what I tried first, and it didn't work at all - the height was even less than one row, probably 0.75 of one row.

> >       <splitter id="detailsDeck-splitter"
> >                 collapse="after"
> >-                persist="state collapsed"
> >-                disabled="true">
> >+                persist="state collapsed">
> We can't do this because the details toggle depends on it.

What is the details toggle, the More/Fewer button at the bottom of the panel? And how does it depend on the splitter being disabled?