Bug Tracker

Opened 16 years ago

Closed 15 years ago

#501 closed bug (wontfix)

modalContent plugin is not modal

Reported by: [email protected] Owned by:
Priority: major Milestone:
Component: plugin Version:
Keywords: modalContent plugin Cc:
Blocked by: Blocking:

Description (last modified by scott.gonzal)

I've demonstrated this in IE 7 and FF 1.5. There are 2 issues that violate modality:

1) The page elements are still keyboard traversable. This is a big problem because users can invoke any of the controls that accept keyboard events by tabbing to them to obtain input focus.

2) If you are on a page that is taller than the viewport (requires vertical scrolling), the modal overlay does not extend to the bottom of the page - it only covers what's currently visible so that when you sroll down the rest of the page is uncovered and available for mouse events.

Change History (4)

comment:1 Changed 16 years ago by anonymous

Well, I fixed the second issue (the easy one) with a simple stylesheet change to the backdrop rendering function. Specifically, in jQuery.fn.modalContent(), change "postition: 'absolute';" to "position: 'fixed';":

jQuery.fn.modalContent = function(css, animation, speed) {

...

Build our base attributes and allow them to be overriden css = jQuery.extend({

position: 'fixed', <<<<< CHANGE <<<<<< left: '0px', margin: '0px', background: '#000', opacity: '.55'

}, css);

...

}

comment:2 Changed 16 years ago by john

Component: coreplugin

comment:3 Changed 16 years ago by gavinmroy

If this is in regard to my modalContent plugin, this ticket should be closed. I stopped developing this plugin in January of 2007 with the following message "Please note that jquery-modalContent has not been updated to work with jQuery 1.1. If you'd like a light weight modal plugin, try jqModal or blockUI."

AFAIK there are no other plugins that used the same name.

comment:4 Changed 15 years ago by scott.gonzal

Description: modified (diff)
need: Review
Resolution: wontfix
Status: newclosed

This plugin is no longer supported as reported by its creator.

Note: See TracTickets for help on using tickets.