Bug Tracker

Opened 10 years ago

Closed 10 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 10 years ago by john

Keywords: 1.7-discuss added

Nominating ticket for 1.7 discussion.

comment:2 Changed 10 years ago by Rick Waldron

+1,

comment:3 Changed 10 years ago by jaubourg

+1,

comment:4 Changed 10 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 10 years ago by timmywil

+1,

comment:6 Changed 10 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 10 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 10 years ago by timmywil

Component: unfiledcore
Priority: undecidedlow
Status: newopen

comment:9 Changed 10 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 10 years ago by addyosmani

+1

comment:11 Changed 10 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 10 years ago by danheberden

+1

comment:13 Changed 10 years ago by scottgonzalez

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

comment:14 Changed 10 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.