Skip to content
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

Our415 87 header nav base #1369

Open
wants to merge 296 commits into
base: master
Choose a base branch
from

Conversation

rosschapman
Copy link

No description provided.

kramduckner and others added 30 commits June 2, 2024 10:06
add tokens, util classes for our415; modify theme
update readme to include env vars
jtcaovan and others added 23 commits July 1, 2024 12:19
Back Navigation/Favicon/Home page Icon
* basic working version of header dropdown (needs a11y and minor style adjustments)

* CustomDropdown a11y improvements, tab and arrow navigability

* add "all categories" list option

* fix keydown navigation slug

* big style edit for custom dropdown

* more styling and dynamic title for category vs search page

* lint errors

* dynamic sr only title for clarity

* Update CustomDropdown.module.scss add vars
* org page style changes for demo

* lint errors
* Add ServiceCard to OrganizationListingPage

* rm ServiceList from index.tsx

* create new reusable ListingInfoSection for service and org pages

* add ListingInfoSection to OrganizationListingPage and remove corresponding css from org-page.scss

* add ListingInfoSection to ServiceListingPage and clean up module scss

* rm title scss on org page

* lint errors

* mobile styles and cleanup

* clean up commented code
* big refactor: create PageWrapper and PageHeader components, pull those components into Service and Org listing detail pages

* remove global css that is now in modules

* Notes -> NotesList

* org page - pull out unnecessary classes and elements, put NotesList in a ListingInfoSection

* delete org-page.scss now that classnames are shared in pagewrapper

* Change Notes -> NotesList

* delete obsolete css and move some around

* lint errors

* rm deleted org page import to main.scss

* minor updates to css and component naming conventions

* Update PageWrapper.tsx

* Update PageWrapper.tsx rm whitelabel import
* big refactor: create PageWrapper and PageHeader components, pull those components into Service and Org listing detail pages

* remove global css that is now in modules

* Notes -> NotesList

* org page - pull out unnecessary classes and elements, put NotesList in a ListingInfoSection

* delete org-page.scss now that classnames are shared in pagewrapper

* Change Notes -> NotesList

* delete obsolete css and move some around

* lint errors

* rm deleted org page import to main.scss

* minor updates to css and component naming conventions

* TableOfContactInfo -> ContactInfoTableRows component (to be utilized in DataTable)

* create DataTable.module.scss and delete global <table> styling, use DataTable in place of <table> elements in ServiceListingPage

* move more table specific css to DataTable.module.scss

* moving around scss to appropriate component module scss file

* lint error

* Rename Datatable.tsx to DataTable.tsx

* change DataTable name to ListingInfoTable
* adding Homepage and dependent interfaces

* fleshes out Homepage interface and dependencies

* updates Homepage to use data from Strapi

* type and type usage cleanup

* prettier fix

* updates about page to use strapi

* removes remaining references to sanity– we are now without sanity

* prettier fix

* linter fixes

* adds TODO for two column content section

* ignore eslint rule img-redundant-alt for variable value

* renames BaseAttributesResponse to BaseDatumAttributesResponse

* sets alt text to var

* reads strapi keys to config example
@richardxia
Copy link
Member

Hi @rosschapman, I was wondering if you mistakenly submitted this PR against ShelterTech's repo instead of Exygy's fork? It looks like there are many commits in this branch that are PR merge commits that don't exist in ShelterTech's repo.

We are happy in general to accept changes that would make your integration smoother, so long as they don't have any negative impact on our main site or our whitelabeled sites, but I'm guessing that was not your intention here.

seanmalbert and others added 6 commits July 10, 2024 09:11
* updates webpack config to allow pointing to saging api

* updates so api/v2 can proxy to staging api

* prettier fix
* only display NotesList section if there are notes

* Edit ListingInfoTable to take in hardcoded rows and add Tags section with table to servicelistingpage

* reusable LabelTag

* LabelTagRows for Tag section

* add LabelTagRows to ServiceListingPage

* NotesList section minor edit, only show when there are notes

* lint errors

* mobile edits to fix layout

* swap old label tags with new LabelTag and make edits for tooltip label

* comment checkpoint

* comment for future PR

* ListingInfoTable minor edit - pull boolean logic in useRowRenderer var
…d Desktop components into discreet renderers (#75)
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.

6 participants