-
Notifications
You must be signed in to change notification settings - Fork 68
Feedback window minor bugs fixed #105
Feedback window minor bugs fixed #105
Conversation
@bmourat, |
|
||
// Since we are mutating array we need to go backwards | ||
for(NSView *subview in [self.subviews reverseObjectEnumerator]){ | ||
if([subview isKindOfClass:[BITActivityIndicatorButton class]]){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think there are a couple of spaces missing here? 🙂
@@ -353,8 +353,10 @@ - (BOOL)canSendMessage { | |||
} | |||
|
|||
- (IBAction)sendMessage:(id)sender { | |||
[self.manager submitMessageWithText:[self.messageText string] andAttachments:self.attachments]; | |||
[self.manager submitMessageWithText:[self.messageText string] andAttachments:[[NSArray alloc] initWithArray:self.attachments]]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you want, a small syntax improvement could be to use [NSArray arrayWithArray:self.attachments];
instead
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks :)
@@ -211,6 +212,15 @@ - (void)drawRect:(NSRect)dirtyRect { | |||
[super drawRect:dirtyRect]; | |||
} | |||
|
|||
- (void)clearAllImageViews{ | |||
|
|||
// Since we are mutating array we need to go backwards |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a)mutating THE array
-> full English sentences, please.
b) which array? Please be specific.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor comments. Otherwise looking good!
Minor bugs fixed:
Bug with recycling images in tableview cells
Bug with attachment list not being cleared after sending message