Skip to content

Fixes and unit test#1

Merged
MathieuDuponchelle merged 1 commit into
MathieuDuponchelle:depthfrom
kudaba:depth
Jun 20, 2017
Merged

Fixes and unit test#1
MathieuDuponchelle merged 1 commit into
MathieuDuponchelle:depthfrom
kudaba:depth

Conversation

@kudaba
Copy link
Copy Markdown

@kudaba kudaba commented Jun 20, 2017

  • Fix indentation
  • Fix text shifting by 1 pixel on hover
  • Add Unit test
  • Build distribution files.

I feel the default implementation shouldn't shift the text on hover

@MathieuDuponchelle
Copy link
Copy Markdown
Owner

@kudaba , nice, but please remove the disted files, as specified by the maintainers they will be built by them

Fix text shifting by 1 pixel on hover
Add Unit test
@kudaba
Copy link
Copy Markdown
Author

kudaba commented Jun 20, 2017

Ah, I missed that part, commit is fixed.

@MathieuDuponchelle MathieuDuponchelle merged commit b8420ec into MathieuDuponchelle:depth Jun 20, 2017
@MathieuDuponchelle
Copy link
Copy Markdown
Owner

@kudaba , thanks a lot, now to see what @afeld thinks :)

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