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

Feature/issue 12 cast and crew #31

Merged
merged 3 commits into from
Oct 16, 2022
Merged

Conversation

thescientist13
Copy link
Contributor

@thescientist13 thescientist13 commented Oct 9, 2022

Related Issue

resolves #12

Screen Shot 2022-10-08 at 9 32 11 PM

Summary of Changes

  1. Created CastAndCrew component
  2. Created Storybook
  3. Added test cases

TODO

  1. Get real avatar images

Obersavations and tracking items (Help Wanted)

  1. Typography (fonts) seem off? font-family looks correct in the browser but what's displayed in the browser doesn't look anything like the designs? - typography not rendering correctly #32
  2. "tailwindify" all inline styles as much as possible (good first issue) - "tailwindify" inline styles #33
  3. Make text-gray-500 and rgba(107, 114, 128) part of the design system (theme.css) and document in storybook - Add standards colors to styleguide and apply to project #34

@thescientist13 thescientist13 added help wanted Extra attention is needed feature New feature or request labels Oct 9, 2022
@thescientist13 thescientist13 self-assigned this Oct 9, 2022

return `
<div class="text-center">
<img
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would probably be good to add loading="lazy" here to these images

@thescientist13 thescientist13 removed the help wanted Extra attention is needed label Oct 9, 2022
@thescientist13 thescientist13 added the documentation Improvements or additions to documentation label Oct 16, 2022
@thescientist13
Copy link
Contributor Author

Tracking the image avatar gathers as part of #44

@thescientist13 thescientist13 marked this pull request as ready for review October 16, 2022 17:28
@thescientist13 thescientist13 force-pushed the feature/issue-12-cast-and-crew branch from 08af55f to 2d3abfd Compare October 16, 2022 18:20
@thescientist13 thescientist13 merged commit a37c918 into v1 Oct 16, 2022
@thescientist13 thescientist13 deleted the feature/issue-12-cast-and-crew branch October 16, 2022 18:26
thescientist13 added a commit that referenced this pull request Oct 25, 2022
* initial version of cast and crew section

* basic styling and theming of the header

* add test cases
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cast and Crew
1 participant