Skip to content
This repository has been archived by the owner on Feb 28, 2021. It is now read-only.

added directors msg #4

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

Conversation

jay4jyotika
Copy link
Collaborator

No description provided.

@anubhavp28
Copy link
Member

@jay4jyotika It is way too simple. I guess, we need something aesthetically pleasing - I was thinking of something like this -
msg

The left half will contain the message from the director (same as it is), the right part will contain the message from the placement officer (with his picture, name). IMO the contact details need not to be included here, as we have a "contact us" section especially, dedicated to this. And of-course, the shades of green is to be replaced with shades of blue. (What you can also do is that, for smaller width displays make the two sections come vertically one after the other). @satyamtg @AnkurKonar @Ratna04priya

Your pull request contains already merged commits, you have to use a separate branch for each pull request, with the master held constant. Well, keep that aside for now. You can continue working on this pull request the same way as before.

@anubhavp28
Copy link
Member

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