thervh70 / ContextProject_RDD

1 stars 0 forks source link

Restructure the ContentController #189

Closed mpsijm closed 8 years ago

mpsijm commented 8 years ago

All methods that hook/unhook from/to DOM are now structured in a more intuitive order.

Please confirm that I didn't make any errors, so that this can be merged :)

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 91.714% when pulling 2b6c49fb0a895cecc467be5cf6e13c47c706437c on _-restructure-content-controller into 276a96fcc908c899337911374780ec6fcc7c3908 on dev.

mdingjan commented 8 years ago

The changes are fine. :) Ready for merging.

Exclaminator commented 8 years ago

LGTM

thervh70 commented 8 years ago

2 reviews passed. Simple PR. Will merge!