Issue with sub menu clashing with text on the webpage. Can we set a white background to the sub menu?

[Richard]

Dear Hannah @HannahFincham

Thanks :)
There is a few things to think about.
The Sub-menu appears because it goes from 'opacity: 0' to 'opacity:1' when you hover.

nav.primary li:hover .sub-menu{
    opacity: 1;
		padding: 10px;
		color: white;
    pointer-events: auto;
}

In your CSS you are addressing 'nav.primary' which is fine but you also need to add some code doing the same addressing 'mobile-nav' ( the Desktop Navigation & Mobile are two seperate Nav's. depending on browser width - one is active and the other is turned off.

CSS

nav.mobile-nav .sub-menu li{
    display: block;
    position: relative;
		padding-bottom: 5px;
}

nav.mobile-nav li:hover .sub-menu{
    opacity: 1;
		padding: 10px;
		color: white;
    pointer-events: auto;
}

However in general you may wish to design the Mobile menu different because the functionality may not be best for users with phones, here i have the sub-menu on the phone active:

There is also issue with it blocking the menu underneath because of position:absolute - When i remove this CSS we get this:

In Conclusion Hannah :)
This is why your were having trouble seeing the Sub-menu - However you may wish to design a little differently to suit the Mobile format.

Hope I have been helpful and let me know your thoughts, have a wonderful day!

Sincerely
Richard

[HannahFincham]

Thanks so much Richard, that works!!

[HannahFincham]

@Richard-K Actually, I spoke too soon! Got the mobile menu to look how I wanted:

But then realised it had compromised the desktop:

Do you think there's a way of achieving the above mobile menu and still keep the desktop like this:

Sorry, wish I could figure it out!

[Richard]

Dear @HannahFincham

There is for sure a way to have desktop and Mobile CSS seperate with media query: this is why they were made :)
https://www.w3schools.com/css/css_rwd_mediaqueries.asp

I think currently you have removed your Mobile CSS?

Talk soon and have a wonderful day :)

Richard

[NAS]

Hey Richard, (messaging on behalf of @HannahFincham here) I've been having a look at this.

But still such a huge novice we're both struggling. I can't even seem to get back to the point where we had the mobile nav working and desktop not working.

Is there anyway you're able to see what it is I'm missing? Is there just a bit of code i can put into mobile CSS? Or do i need to have in both CSS area's.

Sorry to still be asking on this. Thanks for all the advice so far.

[Richard]

This post is deleted!

[Richard]

Dear @NAS Also on behalf of @HannahFincham :)

This is no worries and will try to help as best i can!
Am i correct that the Desktop is how you wish:

The problem lies with the Sub-menu points not working for Mobile? you wish them all to display as a long list? or the Sub menu points should show only when hovering ( keep in mind that 'Hover' doesnt work on Mobiles as they have no 'Mouse' essentially..)

And this is your CSS for Mobile?

.buttonstyle a{
    padding: 15px 20px;
    display: inline-block;
    border-radius: 3px;
    border-style: solid;
    line-height: 1;
    text-decoration: none;
    border-bottom: solid;
    color: black;
    opacity: 1;
    white-space: nowrap;
}

.buttonstyle a:hover{
    background-color: #f0f0f0;
    text-decoration: none;
    border-bottom: solid;
    color: black;
    opacity: 1;
}
.buttonstyle2 a{
    padding: 15px 20px;
    display: inline-block;
    border-radius: 3px;
    border-style: solid;
    line-height: 1;
    text-decoration: none;
    border-bottom: solid;
    color: white;
    opacity: 1;
    white-space: nowrap;
}

.buttonstyle2 a:hover{
    background-color: #f0f0f0;
    text-decoration: none;
    border-bottom: solid;
    color: black;
    opacity: 1;
}

Just wish to understand better, so i can help better ( The thread is quite long now, so unsure of which part needs fixing! :)

Have a wonderful day & talk soon
Richard

[NAS]

Thanks Richard.
you're spot on.

Desktop displaying as is.
And would like to display full list on mobile.

currently mobile is just this

code_text
.buttonstyle a{
    padding: 15px 20px;
    display: inline-block;
    border-radius: 3px;
    border-style: solid;
    line-height: 1;
    text-decoration: none;
    border-bottom: solid;
    color: black;
    opacity: 1;
    white-space: nowrap;
}

.buttonstyle a:hover{
    background-color: #f0f0f0;
    text-decoration: none;
    border-bottom: solid;
    color: black;
    opacity: 1;
}

nav.mobile-nav .sub-menu li{
    opacity: 1;
    display: block;
    pointer-events: auto;
    padding: 10px;
	color: white;
    position: relative;
	padding-bottom: 5px;
}

thanks for your patience

[Richard]

Dear @NAS

Please remove:

nav.mobile-nav .sub-menu li{
    opacity: 1;
    display: block;
    pointer-events: auto;
    padding: 10px;
	color: white;
    position: relative;
	padding-bottom: 5px;
}

From your above Custom Mobile CSS and replace with:

.laynav .sub-menu{
    opacity: 1;
    position: relative;
    pointer-events: auto;
    border-bottom: none;
    width:100%;

    padding-top: 0px !important;
    padding-bottom: 0px;
    padding-left: 10px;
    padding-right: 0px; 
}

.laynav .sub-menu .menu-item{
    padding-bottom:0px;
}

This gives me this appearance:

Hope this helps & have a wonderful day :)

Sincerely
Richard

[NAS]

amazing thank you Richard :D
working perfectly

thank you

[Richard]

Dear @NAS

No worries & best wishes with the website -it looks great :)

Sincerely
Richard