[wp-trac] [WordPress Trac] #41191: Create browse happy type notice for PHP versions
WordPress Trac
noreply at wordpress.org
Tue Apr 24 09:21:32 UTC 2018
#41191: Create browse happy type notice for PHP versions
-------------------------------------------------+-------------------------
Reporter: joostdevalk | Owner: flixos90
Type: enhancement | Status: reopened
Priority: normal | Milestone: 4.9.6
Component: General | Version:
Severity: normal | Resolution:
Keywords: has-patch ui-feedback needs-testing | Focuses:
| accessibility
-------------------------------------------------+-------------------------
Comment (by flixos90):
After recent discussions and the additional feedback and patch from
@johnjamesjacoby, [attachment:41191.12.diff] is an iteration on
[attachment:41191.10.diff] which takes a couple of suggestions (partly
from [attachment:41191.11.2.diff]) into account. Changes made:
* Fix incorrect statement "WordPress has detected that your site is
running on an insecure version of PHP, which means that a faster website
is just a quick update away." by removing the second part of that
sentence. That furthermore makes it a more clear statement, and the aspect
of speed is mentioned in the second paragraph anyway.
* Remove the whole last section that essentially just revolves around the
user needing to click the button. It helps further shortening the copy, as
requested by the design team, and was redundant.
* Left-align button and make it regular-size, as requested by the design
team and in [attachment:41191.11.2.diff]. @johnjamesjacoby We cannot
shorten the button text itself, as "Learn more" makes it too unclear this
link helps with updating PHP.
* If the PHP version is not insecure, but only out-of-date, show the icon
with yellow color, as requested by the design team. This change is to be
future-proof, but note that at this point, it will never be rendered this
way because we show the notice only on 5.2 which ''is'' insecure.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/41191#comment:108>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list