feat: downloaded the images of the technologies and stylize the section#20
Open
eduardor2m wants to merge 1 commit intonavibar-gatsbyfrom
Open
feat: downloaded the images of the technologies and stylize the section#20eduardor2m wants to merge 1 commit intonavibar-gatsbyfrom
eduardor2m wants to merge 1 commit intonavibar-gatsbyfrom
Conversation
Manuel-Antunes
requested changes
Feb 1, 2022
Comment on lines
+12
to
+47
| const cards = [ | ||
| { | ||
| title: 'React', | ||
| description: 'React is a JavaScript library for building user interfaces.', | ||
| image: react, | ||
| }, | ||
| { | ||
| title: 'NodeJS', | ||
| description: | ||
| "Node.js is a JavaScript runtime built on Chrome's V8 JavaScript engine.", | ||
| image: node, | ||
| }, | ||
| { | ||
| title: 'NextJS', | ||
| description: | ||
| 'Next.js is a framework for server-rendered React applications.', | ||
| image: nextjs, | ||
| }, | ||
| { | ||
| title: 'TypeScript', | ||
| description: | ||
| 'TypeScript is a typed superset of JavaScript that compiles to plain JavaScript.', | ||
| image: typescript, | ||
| }, | ||
| { | ||
| title: 'React', | ||
| description: 'React is a JavaScript library for building user interfaces.', | ||
| image: react, | ||
| }, | ||
| { | ||
| title: 'Go', | ||
| description: | ||
| 'TypeScript is a typed superset of JavaScript that compiles to plain JavaScript.', | ||
| image: go, | ||
| }, | ||
| ]; |
| { | ||
| title: 'Go', | ||
| description: | ||
| 'TypeScript is a typed superset of JavaScript that compiles to plain JavaScript.', |
Comment on lines
+58
to
+75
| <Image | ||
| src={cucicon} | ||
| position="absolute" | ||
| top="55%" | ||
| left="17%" | ||
| width="420px" | ||
| height="360px" | ||
| zIndex={-1} | ||
| /> | ||
| <Image | ||
| src={cucicon} | ||
| position="absolute" | ||
| top="10%" | ||
| left="55%" | ||
| width="420px" | ||
| height="360px" | ||
| zIndex={-1} | ||
| /> |
Member
There was a problem hiding this comment.
Add animations for this images
| marginBottom="50px" | ||
| > | ||
| {cards.map(card => ( | ||
| <Box |
Member
There was a problem hiding this comment.
Add design effects from cards here
leticiabsduarte
suggested changes
Feb 1, 2022
There was a problem hiding this comment.
- Add slide up animation to technology logos (it will happens when you first enter on the section).
- Add slide up animations to çuç logos behind technologies
- Correct the background gradient, note that there are different sections and the gradient defines where they begin and end. Pay attention at figma design background and follow exactly how it was done. Take the line divisions off and correct the colours gradient
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related Issue
Proposed Changes
Additional Info
Checklist
Screenshots