Skip to content
This repository was archived by the owner on Jul 11, 2026. It is now read-only.

primera revisión#31

Open
IrisFyD wants to merge 18 commits into
Laboratoria:masterfrom
IrisFyD:master
Open

primera revisión#31
IrisFyD wants to merge 18 commits into
Laboratoria:masterfrom
IrisFyD:master

Conversation

@IrisFyD

@IrisFyD IrisFyD commented Feb 12, 2020

Copy link
Copy Markdown

No description provided.

Comment thread src/index.html Outdated
background-position: center;
}
</style>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Los estilos van en la hoja de estilos, tengamos consistencia.

Comment thread src/index.html Outdated
</ul>
</nav>
</div>
</header>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cuida tu intentación

Comment thread src/index.html
</div>
<div class="contenedor4">

<p>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Estos espacios son innecesarios.

Comment thread src/index.js Outdated
import cipher from './cipher.js';

console.log(cipher);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Los console.log() nos sirven solo para desarrollar, no se suben en los cambios.

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