[wp-trac] [WordPress Trac] #31467: Images should default to not linking

WordPress Trac noreply at wordpress.org
Tue Aug 25 05:21:15 UTC 2015


#31467: Images should default to not linking
---------------------------------+-----------------------------
 Reporter:  melchoyce            |       Owner:  wonderboymusic
     Type:  enhancement          |      Status:  assigned
 Priority:  normal               |   Milestone:  4.4
Component:  Media                |     Version:
 Severity:  normal               |  Resolution:
 Keywords:  make-flow has-patch  |     Focuses:  ui
---------------------------------+-----------------------------

Comment (by azaozz):

 Looking at 31467.diff, there is no point in doing
 {{{
 getUserSetting( 'align', 'none' ) || defaultProps.align
 }}}

 `getUserSetting()` takes two args: the setting's name and default value.
 If it's not set, it will return the default value, and
 `defaultProps.align` can never be reached.

 If we still want to use `defaultProps.align` as default value, can
 probably do:
 {{{
 getUserSetting( 'align', defaultProps.align )
 }}}

 assuming it is set to something.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/31467#comment:27>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list