Skip to content

Conversation

msarit
Copy link
Contributor

@msarit msarit commented Oct 5, 2018

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Documentation Update

Description

Modifies CSS to better align the Select and Remove buttons with the repo titles.

Related Tickets & Documents

The issue #745 motivated this Pull Request.

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

Screenshot of "Before"

Before-Screenshot

Screenshot of "After"

After-Screenshot

Added to documentation?

  • docs.dev.to
  • readme
  • no documentation needed

[optional] What gif best describes this PR or how it makes you feel?

My-Feelings-in-GIF

padding: 7px 9px;
margin-top: 10px;
width: 100px;
/*padding: 7px 9px;*/
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's remove commented-out CSS :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops! Will take care of it right away lol

Copy link
Contributor

@benhalpern benhalpern left a comment

Choose a reason for hiding this comment

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

Looks great. I agree with @Zhao-Andy, let's remove the css comments and I think this is good to go

@benhalpern benhalpern merged commit 5c245bd into forem:master Oct 5, 2018
isogram pushed a commit to isogram/dev.to that referenced this pull request Oct 15, 2018
… "Integrations" (forem#836)

* Syncing with Upstream master

* Dev Env Keys - Algolia and Github

* Pull from upstream master

* Remove commented code
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.

3 participants