Re: [Dynapi-Dev] BorderManager & Highlighter UPDATED - (files attached)

[Raymond I. <xw...@ya...>]

Kevin,

Nice work! It looks great! I have a few suggestions:

1) DynLayer(html,x,y,w,h,color,image) - Some time ago
DynLayer had so many arguments that it was very
difficult to work with. With 2.9 Dan took the
initiative to only use the most commonly used
arguments. I would suggest that we keep only 6
arguments inside the constructor in order to try and 
keep it simple, but since we already have background
color I think it's only fitting to have background
image as part of the constructor. Agree? Can we stop
at seven?

2) Highlighter(x,y,w,h,color,image) -  the same as
above. For Skins() "image" is already part of the
dictionary object.

3) Another name for setBorder2 could be
setSkinBorder()?


Keep up the good work


PS. Have to tested the creation speed of say 100
layers with skinned borders?


--
Raymond Irving


--- Kevin <ke...@ke...> wrote:
> Hello all,
> 
> Raymond, I hope you don't mind me extending you
> ideas. I've had a go at presenting my
> thoughts on borders which are similar to Benoit's.
> Please unpack the attached file in the
> latest dynapi3x root folder/directory. In principle:
> 
> 1) Images in highlighters (just auto repeat
> background image).
> 2) Have 1 content layer + 4 sides + 4 corners + 1
> container.
> 3) Moving container moves all (no group manager
> needed).
> 4) Border2 is DOM2 spec' i.e. border is outside
> element width (border won't clip content).
> 5) Offer flexibility to have different skin widths
> (perhaps someone can complete this?)
> 6) Let transparent gifs with background colors be
> defaulted and configurable (unfinished).
> 
> There is so much more work to do on this I thought
> I'd stop here and see if it's close!
> 
> Could someone change the example to show how to use
> preloaded images from cache
> rather than strings. Does <layer background=img.src
> ..> do it for ns4? These updates
> have been tested on Linux NS4/Mozilla. Please help
> and offer the fix if it doesn't work
> on your browser.
> 
> -
> Kevin.
> 
> 
> > Hello Everyone,
> > 
> > I've completed the BorderManager and Highligther
> > classes as promised
> > 
> > You can see a live demo here:
> > 
> >
>
http://www24.brinkster.com/dyntools/next/examples/dynapi.gui.bordermanager.html
> > 
> >
>
http://www24.brinkster.com/dyntools/next/examples/dynapi.gui.highlighter.html
> > 
> >
>
http://www24.brinkster.com/dyntools/next/examples/dynapi.gui.focusmanager.html
> > 
> > Note: The setBorder() function now requires the
> > BorderManager Class
> > 
> > 
> > Any comments? Can I go ahead and upload these
> changes
> > to CVS?
> > 
> > 
> > PS. Files are attached
> > 
> > --
> > Raymond Irving
> > 
> > 
> > 
> > __________________________________
> > Do you Yahoo!?
> > The New Yahoo! Search - Faster. Easier. Bingo.
> > http://search.yahoo.com

> ATTACHMENT part 2 application/x-zip-compressed
name=border-update.zip



__________________________________
Do you Yahoo!?
The New Yahoo! Search - Faster. Easier. Bingo.
http://search.yahoo.com