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

Multipurpose front-end part update #2659

Merged
merged 40 commits into from
Sep 3, 2019
Merged

Multipurpose front-end part update #2659

merged 40 commits into from
Sep 3, 2019

Conversation

vbaranov
Copy link
Member

@vbaranov vbaranov commented Sep 2, 2019

Closes #2621
Closes #1771
Closes #2641

All visual changes in this PR indicated in parent PR #2637

Motivation

  • Missing tooltips for Block Details page
  • Inconsistent colours of text in error boxes
  • "Jumping" header height on load

Changelog

CSS and Elixir templates updates

Checklist for your PR

…ckscout Page + Added ERC 721 image to the token transfers page(required additional logic)
…ckscout Page + Added ERC 721 image to the token transfers page(required additional logic)
Added internationalization/Removed image block from token transfers page
Re-added internationalization, re-checked all changes on all devices
Merge updates from master to frontend-tasks.
YegorSan and others added 10 commits August 29, 2019 20:00
Changed text color for all error messages boxes for coin_balances page to the required one / Main Blockscout page banner height bug fix and new changes / New tooltips for Block Details page added / Token image added to the Token instance page.
@vbaranov vbaranov added the ready for review This PR is ready for reviews. label Sep 2, 2019
@coveralls
Copy link

coveralls commented Sep 2, 2019

Pull Request Test Coverage Report for Build ecf2133e-6e09-4c15-8ce0-d31c0aad6c6d

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.3%) to 78.91%

Totals Coverage Status
Change from base Build 652d3b20-36fa-4392-bc78-f6ce5a4b0d96: 0.3%
Covered Lines: 5126
Relevant Lines: 6496

💛 - Coveralls

@vbaranov vbaranov merged commit cf6592d into master Sep 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review This PR is ready for reviews.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Different buttons colors Inconsistency in error box text colors Add tooltips to the "Block Details" page
5 participants