Created attachment 16012 [details] [review]
patch

Setting priority->high due to the patch.

marco don't forget to look at this one :)

The patch looks good but I'm a bit worried about the problems it could
create with the wm. Anyway it's worth a try.

I'm not sure if we should save the size of the property dialog per
dialog because the sizing may be more dependent on the keywords number
than on the url.

Saving per url gives the problem that state.xml will grow without
control, I think it's necessary to introduce a "drop me after 30 days
of not usage" mecanism.

I'd say to get in the generic part and discuss the particular
bookmarks thing forward.

> I'm not sure if we should save the size of the property dialog per
> dialog because the sizing may be more dependent on the keywords number
> than on the url.

Well in regards to dialogs, saving their placement is really nice, especially if you end up opening two or three of them at a time. (which i've done sometimes).

anyway here's a link to an article of spatial interfaces and why this is a "good" idea.

http://arstechnica.com/paedia/f/finder/finder-1.html

Created attachment 16086 [details] [review]
here's the patch for the properties window.

Maybe we should save just position in this case ?
Well this comment was mostly to let you know that I was on crack when 
I said a boolean was enough in state api. Better flags, so we can 
decide if we save SIZE/POSITION or both.

Adding the PATCH keyword.

Mass reassigning of Epiphany bugs to epiphany-maint@b.g.o

Comment on attachment 16012 [details] [review]
patch

bitrot

Comment on attachment 16086 [details] [review]
here's the patch for the properties window.

I'm not sure we should do this without a way to clean obsolete entries from the
states file (otherwise it'll grow without bounds).

pong. This is getting rotten.