Skip to content

<object> tag now has an id.#1

Closed
zippy1981 wants to merge 1 commit into
nyroDev:masterfrom
zippy1981:master
Closed

<object> tag now has an id.#1
zippy1981 wants to merge 1 commit into
nyroDev:masterfrom
zippy1981:master

Conversation

@zippy1981
Copy link
Copy Markdown

@nyroDev
Copy link
Copy Markdown
Owner

nyroDev commented Nov 25, 2010

nyroModal v2 is fully stackable. Which means there could be more than one nyroModal open at the same time. Setting an id directly in the code as you did might cause problem becaus ethe same id "nyroModalSwf" will exists multiple time if 2 or more nyroModal containing swf are opened.

An other id might be to set this id regarder the opener by using an attribute or something else.

By the way, the nyromodal.complete will disapear from github. Please use the specific js for the filter to edit it.

Thanks for your pull request anyway!

@zippy1981
Copy link
Copy Markdown
Author

Its really awesome that you make the containers stackable. I actually edited the original file as well as the complete one, but I guess I did not commit it.

Not having an id in an causes flash not to be able to call javascript in IE, so there is a need for object elements to have ids. Would you prefer a patch where a unique id was always specified or one where the user could specify having one?

Also, if I gave a patch to add a static id to the google code version of nyroModal, would you accept that? I don't see a code repository for that version.

@nyroDev
Copy link
Copy Markdown
Owner

nyroDev commented Nov 25, 2010

nyroModal v1 hosted on google code won't be updated anymore I guess.

regarding your 2 propositions, I thinks a mixed solution would be great :

  • Let the ability for the user to manually set the id (with an attribute of the opener or anything you want)
  • if not manually set, add an uniq id

It could be great ;)

@nyroDev
Copy link
Copy Markdown
Owner

nyroDev commented Feb 25, 2011

Update documentation in Readme file
Add swfObjectId settings and use it in filters.swf or create an uniq ID. Closed by 3fafa30
Add recalc settings for nm.resize function to force the recalculation of the modal. Closed by 3fafa30 Closed by 3fafa30
Fix issue with jQuery 1.5 on IE8 (bogus selector in filters.swf). Closed by 3fafa30
Fix stack error when reopening a modal. Closed by 3fafa30
Add initFilters callbacks in order to force filters. Closed by 3fafa30 Closes#37
Add afterReposition and afterUnreposition callbacks.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants