Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

emphasised dpg indicators met #168

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

Neha9849
Copy link
Member

No description provided.

}

.sdgs img{
width:150px
Copy link
Member

Choose a reason for hiding this comment

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

This hard-codes pixels into the design, so these changes will not be responsive based on screen size. This is similar feedback to what I gave in #146 and #147. Instead of adding more custom CSS changes, again I push for using the existing Bootstrap classes, which saves us the effort of defining and testing our own custom classes. Bootstrap has already done a lot of the hard work for us.

@jwflory jwflory added the ?: blocked Progress blocked by another task or issue label Sep 16, 2022
@jwflory
Copy link
Member

jwflory commented Oct 7, 2022

These changes could fit in the Team Profiles section after the CSS reorganization in PR #179:

/* Team Profiles: start *******************************************************/
.startupCard {
align-items: center;
display: flex;
flex-direction: column;
justify-content: space-between;
}
/* Team Profiles: end *********************************************************/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
?: blocked Progress blocked by another task or issue T: improvement Improves on something that already exists
Projects
Status: Waiting on external
Development

Successfully merging this pull request may close these issues.

2 participants