Closed AYastrebov closed 10 years ago
Please, be careful with updating. I've included mailcore2 and apptentive-ios as a git-submodules. I think it is a more better solution of using side iOS libraries which are rapidly developed (like mailcore2).
If you don't like this approach, feel free to let me know about it.
Sounds fine. Been meaning to do it, but we should add a build script that grabs the submodules if you don't have it.
I'd like to keep this easy to use for first time developers and git submodules is not common knowledge. (not hard, just unnecessary friction)
Ideally, it would be something nicer then the default uialertview. We should be using the flat uialertview like we do with the logout dialog:
http://cl.ly/image/070R0z0t1H47 https://github.com/Grouper/FlatUIKit#alert-views
Given that the auth screen (http://cl.ly/image/1b0b271c0p0C) is the first thing people see of that inbox, we'd like it to look good. So the header should look nicer (flat bar and arrows).
Here's a mockup.
Color for header bar: f1f1f1 Color for header text: 565656 Color for back button: 2980b9 (belizehole in uicolor+flatui.h)
No need for a forward button. The back button shouldn't show up on the login page. Should only show up on other pages, like the can't access your account page.
When trying to scroll horizontal, you end up trigger the right hand menu. We should figure out how to drag on the message body while leaving the drag possible on the message list.
http://cl.ly/image/2D0C3z3d0c2D
Also, it should only scroll the message, not the headers
It'll be easier if these are all separate pull requests.
I know I had earlier commented that we should figure out how to make the git submodules autoload so it's easier. As it turns out, much harder then I expected. I tried a bunch of methods, including putting it as a preaction in the scheme, but I think it turned out that trying to get it to be magical was harder, confusing, and more error prone then just letting the user know about it.
Anyways, we should add this run script to thatinbox (unless you can think of something slicker, which would be awesome!) http://cl.ly/image/2R3l2W2X3m0M
# A quick test for if submodules exist. Not a perfect test, but should work for 99% of the cases
if [ ! -d "Vendor/mailcore2" ] || [ $(ls -A Vendor/mailcore2) ]
then
echo 'error: git submodules not present.'
echo 'To fix this: just run following commands in the terminal'
echo ''
echo '> cd '${PROJECT_DIR}
echo '> git submodule init'
echo '> git submodule update'
exit 1
fi
Thanks for all the help. Super excited to be able to push out these fixes. Let me know how I can help (sorry about the delay on the review; i'll be much faster next time, promise).
Please, take a look. The run script doen't work for me. I have all submodules updated, but it fails with error. We should figure out how to fix that.
Ah. nvm. looks like you updated flatuikit. got it.
Let's not worry about the build script in this pull request. already getting a bit too big. heh.
Forgot that I can't push to this branch..... anyways, the changes are pretty straightforward.
Not sure why your hex code color stuff is funky looking: http://cl.ly/image/2v3G3X2d0s3H Changed to use flatui which seems to fix it.
#import "AuthNavigationViewController.h"
-#import "UIColor+FromHex.h"
+#import "UIColor+FlatUI.h"
#import "UINavigationBar+FlatUI.h"
#import <QuartzCore/CALayer.h>
@@ -73,7 +73,7 @@
_activityIndicator.hidden = YES;
// Configure FLATUI NavigationBar
- [self.navigationBar configureFlatNavigationBarWithColor:[UIColor colorWithHexCode:@"f1f1f1"]];
+ [self.navigationBar configureFlatNavigationBarWithColor:[UIColor colorFromHexCode:@"F1F1F1"]];
index 6c42b5d..47085c6 100644
--- a/App/Menu/MenuViewController.m
+++ b/App/Menu/MenuViewController.m
@@ -235,7 +235,7 @@ bool firstLoad = YES;
#pragma mark Logout Functions
- (void)alertView:(FUIAlertView *)alertView clickedButtonAtIndex:(NSInteger)buttonIndex {
- if (buttonIndex == 0){
+ if (buttonIndex == 1){
[self logoutPressed];
}
One thing I didn't fix. http://cl.ly/image/1L1C2p410V44
What's going on?
Also, thanks for the work. Gave you push pull rights for the repo so once you finish these minor fixes, feel free to marge. When I write code for this, i'll ask you for a pull request and code review and vice versa.
Sorry for ugly alert, my mistake.
Fixed issue #6
Kind regards, AgileFusion