Closed d4mation closed 6 years ago
Based on the title of the section in the Punchlist Document, this seems to be for the Testimonials on the Home Page.
The Home Page Testimonials are technically already a Slider, so that much is already implemented as part of Joel's design. It just doesn't have it's Next/Previous buttons as overlaid arrows as would be preferred.
@evasolo @marieepicraces Am I understanding this correctly?
If so, I'll work on moving them to arrow buttons rather than what is there now. I'll also see what I can do about having it load a different set of 3 for each page load, but randomness can be tricky as it often isn't directly supported due to performance/caching reasons.
Additional information from the email conversation between Eva and I:
I completely understand what you're saying and in most website cases I agree with you, but signing up for a race is a unique situation. It is not a site that folks typically go back to over and over again before signing up for a race. We have about 30 sec - one minute to engage them to want to learn more as opposed to a website where somebody is looking for store hours, or news.
I don't mind if there are arrows as opposed to automatic sliding, but I don't like the "next" and "previous" buttons below.
We would like arrows
Yes, they should be coming, instead of the buttons underneath.
OK! I am good with the photo uploads.
This is now on Staging.
@evasolo This need approval on Staging before it can be pushed Live.
I thought I had already approved the testimonials. They are good to be pushed live.
Be fit, be well, and keep smiling,
Eva Solomon Founder and CEO Epic Races, LLC
p: 734.585.7101 ext. 700 m: 734.678.5045 a: 600 South Wagner, Ann Arbor, MI 48103 w: epicraces.com https://www.epicraces.com/ e: eva@epicraces.com
On Thu, Sep 20, 2018 at 1:53 PM, Eric Defore notifications@github.com wrote:
@evasolo https://github.com/evasolo This need approval on Staging before it can be pushed Live.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/realbig/RBM_TH_EPIC/issues/138#issuecomment-423275496, or mute the thread https://github.com/notifications/unsubscribe-auth/APinaswih0QuhGPTnDOie92DDiRZCc_uks5uc9YugaJpZM4WlfZk .
This is now Live
From Document:
From Email:
@BigActual's reply:
@evasolo's reply:
My Notes/Suggestions to follow