Skip to content

Conversation

@craigsteel
Copy link

No description provided.

@cmccrorie
Copy link
Member

cmccrorie commented Mar 8, 2021

@craigsteel
Hi Craig Just looked at the pull request and tested it. I have found some issues with this.
The first one is that when clicking on the "About" "Services" and "Exist-DB" tabs It does not respond unless I go to the "Careers" or "Contact" tab / page first then go back it will then work.
Seen in below screen shot.

Image from iOS

The second issue I found was that you can see the tabs from the main pages along the side of the menu screen. This is also followed by some text appearing at the bottom of the screen occasionally - this almost makes the screen look like it is glitching. Seen in the Screen shot below.

Image from iOS (1)

Lastly, The menu icon covers up parts of the text when scrolling down pages which can become annoying to users of the site.
Seen in the screen shot below.

Image from iOS (2)

@craigsteel
Copy link
Author

craigsteel commented Mar 8, 2021 via email

@cmccrorie
Copy link
Member

@craigsteel
Hi,
Okay no worries thank you I will retest it when you have uploaded it.
Thank you .

@craigsteel
Copy link
Author

craigsteel commented Mar 8, 2021 via email

@cmccrorie
Copy link
Member

cmccrorie commented Mar 11, 2021

Hi @craigsteel
The last changes look great thank you, I have tested it and all working great. However If at all possible on some of the screens at the bottom for the mobile site there is a black menu that I feel does not need to be there now with how the new menu works and just looks out of place. Could this be taken off the mobile site at all?
Thank you for your time, I have attached a screenshot of what I am on about.
Image from iOS
Kind Regards
Callum

@craigsteel
Copy link
Author

craigsteel commented Mar 13, 2021 via email

@cmccrorie
Copy link
Member

@craigsteel
Hi Craig
The changes that you have made all look good and work well. Thank you for sorting it all out.
Kind Regards
Callum

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants