Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

Close button on modals

#7

Is it possible to include a close button in the Modal window? I know its not required for us geeks, but an ordinary user will be lost without a close button...

Reported by Suneth Mendis · June 16th, 2008 @ 06:15 PM

State: hold
Milestone: none
Assigned to: Ryan Johnson Ryan Johnson

Activity

  1. Caleb Jones
    Caleb Jones

    I would vehemently oppose automatically putting a close button in modal windows. Control (form a programmer's perspective) is one of the reasons I use livepipe's modal implementation.

    That said, I'm not opposed to the addition of a parameter that would automatically generate a close button. That way those who want complete control can have it, and those who want a canned solution can do so (best of both worlds).

    June 20th, 2008 @ 08:39 AM

  2. Ryan Johnson
    Ryan Johnson
    • State changed from unconfirmed to confirmed

    Duly noted and agreed. Maybe something like this syntax:

    new Control.Modal({

    closeOnClick: new Element('div')

    });

    ?

    June 20th, 2008 @ 10:58 AM

  3. Suneth Mendis
    Suneth Mendis

    Hi Caleb,

    I didn't mean to have a close button always present on Modals. That would be defeat the whole purpose of using Modal control. All I wanted to have an option to include a close, should there is a need; like the way I use Modal in my application. (And obviously after some feedback from very average users.

    Ryan, that's exactly what i would like. Thanks mate!

    June 20th, 2008 @ 11:37 PM

  4. Suneth Mendis
    Suneth Mendis

    Forgive me for all the typos!

    June 20th, 2008 @ 11:50 PM

  5. Nathan L Smith
    Nathan L Smith
    • Tag set to control.modal
    • Milestone cleared.

    Moving this feature to 1.1

    May 18th, 2009 @ 04:13 PM

  6. Nathan L Smith
    Nathan L Smith
    • Tag changed from control.modal to control.modal, feature

    May 18th, 2009 @ 04:14 PM

  7. Nathan L Smith
    Nathan L Smith
    • State changed from confirmed to hold

    August 9th, 2009 @ 07:02 PM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.