Context Navigation

← Previous Ticket
Next Ticket →

#38392 closed defect (bug) (fixed)

Twenty Seventeen: Display 'scroll down' arrow alternative when there is no menu

Reported by:	laurelfulford	Owned by:	karmatosed
Priority:	normal	Milestone:	4.7
Component:	Bundled Theme	Version:
Severity:	normal	Keywords:	has-patch needs-testing
Cc:		Focuses:

Description

From a comment by melchoyce https://github.com/WordPress/twentyseventeen/pull/458#issuecomment-254592939:

"I'm wondering if there’s any way we could make the scrollability more obvious, maybe by adding a down arrow over the header image so there’s some indication that you can scroll more. Or not taking up the whole viewport, so a little bit of the page background peeks through at the bottom."

Attachments (4)

38392.patch (2.0 KB ) - added by nnaimov 10 years ago.: Move the scroll down button into the header if there is no menu set.
38392.2.patch (2.8 KB ) - added by davidakennedy 10 years ago.: Style and spacing adjustments
down-arrow-baseline.png (282.3 KB ) - added by melchoyce 10 years ago.
38392.3.patch (2.9 KB ) - added by davidakennedy 10 years ago.: Adjust arrow position. See: https://cldup.com/PqdG72MpqT.png

Download all attachments as: .zip

Change History (12)

@nnaimov
10 years ago

Attachment 38392.patch added

Move the scroll down button into the header if there is no menu set.

#1 @davidakennedy
10 years ago

Keywords has-patch needs-testing added; needs-patch removed

@davidakennedy
10 years ago

Attachment 38392.2.patch added

Style and spacing adjustments

#2 @davidakennedy
10 years ago

38392.2.patch builds off the previous patch and adjusts the styling for the color when the skip link is not in the nav. It also tweaks some spacing issues.

@melchoyce Do you want to adjust spacing slightly? @laurelfulford How does this look to you?

@nnaimov Thanks for the patch! In the future, please create patches from the root of your WordPress install. It makes easier for others to apply your patches. :)