Bug Tracker

Opened 12 years ago

Closed 12 years ago

#9401 closed enhancement (wontfix)

Better Hooks Organization

Reported by: Rick Waldron Owned by:
Priority: low Milestone: 1.7
Component: core Version: 1.6.1
Keywords: Cc:
Blocked by: Blocking:

Description

Is it too late to change all these "fooHooks" into something a little more organized, like... https://gist.github.com/973761

Change History (14)

comment:1 Changed 12 years ago by john

Keywords: 1.7-discuss added

Nominating ticket for 1.7 discussion.

comment:2 Changed 12 years ago by Rick Waldron

+1,

comment:3 Changed 12 years ago by jaubourg

+1,

comment:4 Changed 12 years ago by john

+1, Sounds great, as long as we keep the original hook points as well (maybe we can have special events in here as well?).

comment:5 Changed 12 years ago by Timmy Willison

+1,

comment:6 Changed 12 years ago by paul.irish

-1, straight up yak shaving. no benefit to users. (I don't think anyone will be iterating over all hooks)

comment:7 Changed 12 years ago by Rick Waldron

@paul.irish the proposal isn't meant to directly benefit the users, it's primarily for improving maintenance and organization of internal code.

comment:8 Changed 12 years ago by Timmy Willison

Component: unfiledcore
Priority: undecidedlow
Status: newopen

comment:9 Changed 12 years ago by dmethvin

-1, This seems like rearranging the sock drawer to me, is there any pro to balance the con of going back and fixing all the existing code?

comment:10 Changed 12 years ago by addyosmani

+1

comment:11 Changed 12 years ago by ajpiano

+0, It seems like all these "fooHooks" are just kind of ugly looking, but on the other hand, at least they give primacy to the thing that's being manipulated - "css" comes before "hooks" - whereas with the object of jQuery.hooks, there's just this abstract concept of "hooks" - which most people thought jQuery had long before we ever had any. (Heard the phrase "jQuery hooks into the document ready event" recently enough?) So it kind of makes it more confusing for people who need to use it, but yeah, it certainly is less than "appealing" as is...

comment:12 Changed 12 years ago by danheberden

+1

comment:13 Changed 12 years ago by scottgonzalez

-1, I fail to see how this is any more organized

comment:14 Changed 12 years ago by Rick Waldron

Keywords: 1.7-discuss removed
Milestone: 1.next1.7
Resolution: wontfix
Status: openclosed
Note: See TracTickets for help on using tickets.