Skip to content
This repository was archived by the owner on Dec 2, 2020. It is now read-only.

Fix several UI bug when displaying data with negative numbers#79

Merged
Boris-Em merged 7 commits intoBoris-Em:masterfrom
ben181231:master
Nov 2, 2014
Merged

Fix several UI bug when displaying data with negative numbers#79
Boris-Em merged 7 commits intoBoris-Em:masterfrom
ben181231:master

Conversation

@ben181231
Copy link
Copy Markdown
Contributor

  • Fix the method calculating max value of the data
  • Fix the Y Axis Label Offset when displaying nagative numbers
  • Fix the Pop Up Report when displaying nagative numbers

Ben Lei added 3 commits October 7, 2014 14:05
- Fix the method calculating max value of the data
- Fix the Y Axis Label Offset when displaying nagative numbers
- Fix the Pop Up Report when displaying nagative numbers
@Boris-Em
Copy link
Copy Markdown
Owner

Thank you for your contribution @ben181231. I haven't find much time to work on BEMSimpleLineGraph lately, but as soon as I do so, I'll be taking a look at your PR and hopefully merge it!

Ben Lei added 2 commits October 15, 2014 16:09
# Please enter the commit message for your changes. Lines starting
# with '#' will be kept; you may remove them yourself if you want to.
# An empty message aborts the commit.
# On branch master
# Your branch is up-to-date with 'origin/master'.
#
# Changes to be committed:
#	modified:   Classes/BEMSimpleLineGraphView.m
#
@skywinder
Copy link
Copy Markdown
Contributor

@ben181231 Thanks for your pull-request. I started to doing same fixes, before Iooked at your pull-request. But, what's wrong with Travis build? It's seem that you broke something in this pull-request.

@ben181231
Copy link
Copy Markdown
Contributor Author

I just had a look on the fails on travis. Seems like the travis was updated to support new Xcode and new SDKs. However, the xctool is not yet supporting that.

@skywinder
Copy link
Copy Markdown
Contributor

Got it. yes, it's not your fault. Look at pull-request #84 and feel free to use it!

update travis script to avoid "exited with 134"
skywinder added a commit to skywinder/BEMSimpleLineGraph that referenced this pull request Oct 27, 2014
skywinder added a commit to skywinder/BEMSimpleLineGraph that referenced this pull request Oct 27, 2014
skywinder added a commit to skywinder/BEMSimpleLineGraph that referenced this pull request Oct 27, 2014
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Looks good.
Although, should be set to YES in the sample project.
The comment should state that the label will be at the bottom of the graph and not on the side.

Boris-Em added a commit that referenced this pull request Nov 2, 2014
Fix several UI bug when displaying data with negative numbers
@Boris-Em Boris-Em merged commit dec9c37 into Boris-Em:master Nov 2, 2014
@Boris-Em
Copy link
Copy Markdown
Owner

Boris-Em commented Nov 2, 2014

Thank you so much for your valuable contribution @ben181231 (and @skywinder). I will take care of the note that I added.
Sorry it took me so long to merge this PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants