-
Notifications
You must be signed in to change notification settings - Fork 103
Website redesign #348
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Website redesign #348
Conversation
b360977
to
06c398c
Compare
<input id="main-search-input" | ||
type="text" | ||
aria-label="Search" | ||
class="form-control st-default-search-input" | ||
class="search-bar-input form-control" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Check to see if the swift type works I doubt it would fail with the css change but probably requires some verification would be required as the js provides accessible search through the dynamic widget.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I can tell, the class change doesn't affect the functionality (the scripts appear to reference the main-search-input
ID, and the search results seem the same), but if there's something I'm missing, do let me know.
This reverts commit bb7aee4.
Summary of Changes
Design changes have been made in accordance with the wireframes and follow-up discussion during the OData team meeting.
Before & After
Home:

Get Started:

Tools:

Libraries:

Blog:

Ecosystem:

Remaining Work
Remaining tasks (such as those regarding the UI and data storage) detailed in handoff documentation. Of particular note is hooking up the contribution form, which is marked via a TODO in comments in this PR.