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

Update Hero Design & General Improvements #17

Closed
wants to merge 7 commits into from
Closed

Conversation

diogogmatos
Copy link
Member

@diogogmatos diogogmatos commented Sep 27, 2023

  • Improved the code for better modularity and focused on reducing the use of repeated code.

  • Better documented and organized existing code for better readability and easier understanding.

  • Made some tweaks to the UI.

Update:

  • Updated the design to @EnzoVieira 's and added a Carousel that allows to scroll between different elements.

image

@netlify
Copy link

netlify bot commented Sep 27, 2023

Deploy Preview for cesium-hydrogen-new ready!

Name Link
🔨 Latest commit 0b6a576
🔍 Latest deploy log https://app.netlify.com/sites/cesium-hydrogen-new/deploys/6515f758e8a87a0008c1f3f9
😎 Deploy Preview https://deploy-preview-17--cesium-hydrogen-new.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@diogogmatos diogogmatos self-assigned this Sep 27, 2023
@diogogmatos diogogmatos added the enhancement New feature or request label Sep 27, 2023
@diogogmatos diogogmatos requested review from EnzoVieira, afonsopimenta, ruilopesm and GresosJ and removed request for EnzoVieira September 27, 2023 23:26
@ruilopesm ruilopesm changed the title Improve & Document Code refactor: improve and document code Sep 28, 2023
@diogogmatos diogogmatos changed the title refactor: improve and document code Update Hero Design & General Improvements Sep 28, 2023

export default function Team() {
export default function Equipa() {
Copy link
Member

Choose a reason for hiding this comment

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

It is a bit weird to have the naming in portuguese 😅
I think the page should be called team and so the component.

Copy link
Member

Choose a reason for hiding this comment

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

Agreed, i think it is good having the navbar link in portuguese since the whole website is, but the component and filename (therefor, the url) should stay in english.

@diogogmatos
Copy link
Member Author

Closed due to integral project changes both in planning and structure, #21

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants