Skip to content
This repository was archived by the owner on Jun 7, 2024. It is now read-only.

Initial layout #13

Merged
merged 19 commits into from
Sep 20, 2021
Merged

Initial layout #13

merged 19 commits into from
Sep 20, 2021

Conversation

Phosphorus-M
Copy link
Member

Agregado en la landing page:

  • Header
  • Hero
  • Acerca
  • Nuestros proyectos

En teoría todo es responsive de momento, faltan varias cosas pero bueno, es un comienzo.

El diseño que se viene usando es este desde luego esta abierto a cambios y sugerencias.

@LeoBorai
Copy link
Member

LeoBorai commented Jun 7, 2021

@Phosphorus-M I think it would be nice to have screenshots here, so we can see what's being merged

@LeoBorai
Copy link
Member

LeoBorai commented Jun 7, 2021

@Phosphorus-M please fix GitHub actions clippy and fmt issues, its important to keep our codebase as clean as possible, remember to run both commands before pushing to GitHub.

cargo fmt && cargo clippy

@LeoBorai LeoBorai requested a review from a team June 7, 2021 00:45
@@ -0,0 +1,65 @@
const path = require('path');
Copy link
Member

Choose a reason for hiding this comment

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

I think webpack is too much for this, why not using PostCSS along with Rollup? Is faster and lightweight.

@@ -0,0 +1,65 @@
const path = require('path');
const WasmPackPlugin = require('@wasm-tool/wasm-pack-plugin');
Copy link
Member

Choose a reason for hiding this comment

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

Is this required? We are using Trunk already so Im not sure why we have a WASM loader here.

@LeoBorai
Copy link
Member

@Phosphorus-M please make sure you rebase! It makes easier PR reviewing.

@LeoBorai
Copy link
Member

The router looks good so far, but I believe we will encounter issues with client side routing given that we are using GitHub Pages which (AFAIK) it doesn't support client-side routing.

We would have to either use a hash router for this or change our hosting provider to make it work as expected.

@Phosphorus-M Phosphorus-M merged commit 05202fb into development Sep 20, 2021
@Phosphorus-M Phosphorus-M deleted the Initial-Layout branch September 20, 2021 19:30
@Phosphorus-M Phosphorus-M restored the Initial-Layout branch September 20, 2021 19:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants