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

Jkmarx/data set 2 fix button hide #1804

Merged
merged 3 commits into from Jun 16, 2017

Conversation

jkmarx
Copy link
Member

@jkmarx jkmarx commented Jun 16, 2017

  • Fix button hiding behind launch pad
  • Add icon to tool selection menu
    fix-hide-button

@jkmarx jkmarx added this to the Release 1.5.7 milestone Jun 16, 2017
@jkmarx jkmarx self-assigned this Jun 16, 2017
@jkmarx jkmarx changed the base branch from develop to jkmarx-data-set-2-bound-bug June 16, 2017 16:12
@codecov-io
Copy link

codecov-io commented Jun 16, 2017

Codecov Report

Merging #1804 into jkmarx-data-set-2-bound-bug will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@                     Coverage Diff                      @@
##           jkmarx-data-set-2-bound-bug    #1804   +/-   ##
============================================================
  Coverage                        40.39%   40.39%           
============================================================
  Files                              388      388           
  Lines                            25339    25339           
  Branches                          1271     1271           
============================================================
  Hits                             10236    10236           
  Misses                           15103    15103

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d2b9562...7e25bd9. Read the comment docs.

@jkmarx jkmarx merged commit 2eec978 into jkmarx-data-set-2-bound-bug Jun 16, 2017
@jkmarx jkmarx deleted the jkmarx/data-set-2-fix-button-hide branch June 16, 2017 16:24
jkmarx added a commit that referenced this pull request Jun 16, 2017
* Fix bound bug by using the auto-resize ui-grid module.

* Remove unused dependency.

* Fix unit test.

* Jkmarx/data set 2 fix button hide (#1804)

* Show tool icon when selected.

* Fix hidden button bug.

* Add styles.
jkmarx added a commit that referenced this pull request Jun 16, 2017
* Fix parsing error.

* Fix bound bug by using the auto-resize ui-grid module. (#1802)

* Fix bound bug by using the auto-resize ui-grid module.

* Remove unused dependency.

* Fix unit test.

* Jkmarx/data set 2 fix button hide (#1804)

* Show tool icon when selected.

* Fix hidden button bug.

* Add styles.
jkmarx added a commit that referenced this pull request Jun 16, 2017
* Remove unused method.

* Fix parsing error. (#1801)

* Fix parsing error.

* Fix bound bug by using the auto-resize ui-grid module. (#1802)

* Fix bound bug by using the auto-resize ui-grid module.

* Remove unused dependency.

* Fix unit test.

* Jkmarx/data set 2 fix button hide (#1804)

* Show tool icon when selected.

* Fix hidden button bug.

* Add styles.
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.

None yet

2 participants