Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

API changes #101

Closed
paulirish opened this issue Aug 13, 2010 · 4 comments
Closed

API changes #101

paulirish opened this issue Aug 13, 2010 · 4 comments

Comments

@paulirish
Copy link
Member

There are two properties that are particularly verbose and I'd love to see shorter:

Modernizr.crosswindowmessaging ==> Modernizr.postmessage
Modernizr.historymanagement ==> Modernizr.history

Thoughts? (Faruk?) Not worth it?
@KuraFire
Copy link
Member

Having them as shorter properties is something I'm okay with, but what's your suggestion w.r.t. backwards compatibility? Break it, or have two properties for each?

@paulirish
Copy link
Member Author

Two props, deprecating one.
Removing it at 2.0.

@KuraFire
Copy link
Member

That sounds good. We announce that we'll remove them entirely with 2.0.

@paulirish
Copy link
Member Author

rename historymanagement and crosswindowmessaging properties. deprecated them.

closed by 3003224

ralphholzmann pushed a commit to ralphholzmann/Modernizr that referenced this issue Jun 8, 2011
@paulirish
rename historymanagement and crosswindowmessaging properties. depreca…
3003224 
…ted them.

fixes Modernizr#101
patrickkettner pushed a commit to patrickkettner/Modernizr that referenced this issue Feb 22, 2015
@paulirish
rename historymanagement and crosswindowmessaging properties. depreca…
82c13ce 
…ted them.

fixes Modernizr#101
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@KuraFire @paulirish