[wp-trac] [WordPress Trac] #21368: Welcome screen design - v2

WordPress Trac wp-trac at lists.automattic.com
Wed Aug 29 02:43:19 UTC 2012


#21368: Welcome screen design - v2
-------------------------+------------------
 Reporter:  lessbloat    |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  3.5
Component:  General      |     Version:
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |
-------------------------+------------------

Comment (by georgestephanis):

 Latest revision removes the install_theme capability check, as that
 tooltip shouldn't show in a walkthrough, as the link kicking it off got
 removed!  Also removed the relevant function, etc.

 Added in a get variable appender for the links we care about.  This way we
 can only show the step 2 tooltip when following a click on the link from
 the previous page.  No cookies needed!

 Also made a minor change in verbiage as requested by Koop (element becomes
 link)

 Koop requested changing setup to use e.preventDefault() rather than return
 false -- I tried it, but it won't work, as the same function also needs to
 run on pageload, and that generates a script error in practice.  Has to
 return false to prevent the link from going, either that or some
 convoluted test for if e is an event then preventDefault()-ing.

 Also removed the media block from colors-fresh as per Helen.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/21368#comment:19>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list