Skip to content

Create Login_signup#43

Open
davidbranco2 wants to merge 1 commit into
mainfrom
davidbranco2-patch-1
Open

Create Login_signup#43
davidbranco2 wants to merge 1 commit into
mainfrom
davidbranco2-patch-1

Conversation

@davidbranco2

Copy link
Copy Markdown
Collaborator

No description provided.

@schultzemn schultzemn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The html looks great, but the css doesn't follow our overall styling and the javascript should be in a separate file and imported.

</div>
</div>

<script>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Javascript should be separated from the html.

<meta name="viewport" content="width=device-width, initial-scale=1.0" />
<title>Budget App Login</title>
<style>
* {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We need to have a unified style for the whole site, which is in '../static/styles.css'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants