Skip to content
This repository has been archived by the owner on Dec 27, 2020. It is now read-only.

Ui respo #31

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Ui respo #31

wants to merge 3 commits into from

Conversation

MightyMoin
Copy link
Collaborator

Fixes #25 (<=== Replace 0000 with the Issue Number)
Made admin UI complete responsive.
Screenshot (134)
Screenshot (132)
Screenshot (133)

Copy link
Member

@rishabhshuklax rishabhshuklax left a comment

Choose a reason for hiding this comment

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

Some changes but most of this looks good

<div class="inp-label">
<span> Enter the email </span>
<span class="lappie"> Enter the email </span>
Copy link
Member

Choose a reason for hiding this comment

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

Use appropriate class names

<img style="width: 100%; height: 100%;" src="../../images/profile_data.svg" alt="">
</div>
<div style="width: 40%;">
<div style="width: 40%;" class="info-width">
<h2>Basic Details:</h2>
<div><strong>Name:&nbsp;</strong><%=user.name%></div><br>
<div><strong>Email:&nbsp;</strong><%=user.email%></div><br>
Copy link
Member

Choose a reason for hiding this comment

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

Also, move all inline css to .css file

@MightyMoin
Copy link
Collaborator Author

MightyMoin commented Aug 25, 2020

resolved conflicts and updated with master with responsive.

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.

Make the site responsive
2 participants