[buddypress-trac] [BuddyPress Trac] #7734: BP-Nouveau Group Creation Previous-Next navigation: Links vs Buttons
buddypress-trac
noreply at wordpress.org
Fri Jun 29 10:04:43 UTC 2018
#7734: BP-Nouveau Group Creation Previous-Next navigation: Links vs Buttons
---------------------------------------+---------------------
Reporter: mercime | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: 4.0
Component: Templates | Version:
Severity: normal | Resolution:
Keywords: has-screenshots has-patch |
---------------------------------------+---------------------
Changes (by DJPaul):
* milestone: Under Consideration => 4.0
Comment:
Sorry no-one gave you feedback on this sooner, @mercime.
Both variants of the patch have translation problems. e.g. `esc_html__`
instead of `esc_html`, and the patch with the arrow, that arrow needs to
be included in the translated string for RTL languages. I am also wary of
styling links to look like buttons because that's hard to do reliably
cross-browser. It seems that the button element is not appropriate here,
either, which is annoying, as that would have solved the visual
appearance.
For the arrow patch, I think it just looks bad to have two different types
of navigation style next to each other like that.
I think if we can change the layout of these navigation buttons, then
perhaps the arrow patch will be better. I don't have any time to play
around with better button placement, but I would revisit the never-
questioned-decision about whether the previous/next wizard buttons need to
be next to each other.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7734#comment:8>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list