diff options
-rw-r--r-- | macosx/Controller.m | 18 | ||||
-rw-r--r-- | macosx/HBAdvancedController.h | 4 | ||||
-rw-r--r-- | macosx/HBAdvancedController.m | 8 | ||||
-rw-r--r-- | macosx/HBAudioController.h | 5 | ||||
-rw-r--r-- | macosx/HBAudioController.m | 12 | ||||
-rw-r--r-- | macosx/HBChapterTitlesController.h | 6 | ||||
-rw-r--r-- | macosx/HBChapterTitlesController.m | 10 | ||||
-rw-r--r-- | macosx/HBPresetsViewController.m | 16 | ||||
-rw-r--r-- | macosx/HBSubtitlesController.h | 4 | ||||
-rw-r--r-- | macosx/HBSubtitlesController.m | 12 | ||||
-rw-r--r-- | macosx/HBVideoController.h | 6 | ||||
-rw-r--r-- | macosx/HBVideoController.m | 14 | ||||
-rw-r--r-- | macosx/HBViewValidation.h | 2 |
13 files changed, 52 insertions, 65 deletions
diff --git a/macosx/Controller.m b/macosx/Controller.m index 83d7d06c0..3ba09b792 100644 --- a/macosx/Controller.m +++ b/macosx/Controller.m @@ -725,22 +725,14 @@ static NSString * ChooseSourceIdentifier = @"Choose Source It } [controls[i] setEnabled: b]; } - - - if (b) - { - /* we also call calculatePictureSizing here to sense check if we already have vfr selected ??? */ - [self pictureSettingsDidChange]; - } - [fPresetsView setEnabled:b]; - [fVideoController enableUI:b]; - [fChapterTitlesController enableUI:b]; - [fSubtitlesViewController enableUI:b]; - [fAudioController enableUI:b]; + [fPresetsView setUIEnabled:b]; + [fVideoController setUIEnabled:b]; + [fAudioController setUIEnabled:b]; + [fSubtitlesViewController setUIEnabled:b]; + [fChapterTitlesController setUIEnabled:b]; } - /*********************************************************************** * updateDockIcon *********************************************************************** diff --git a/macosx/HBAdvancedController.h b/macosx/HBAdvancedController.h index cf077b448..d80923ddd 100644 --- a/macosx/HBAdvancedController.h +++ b/macosx/HBAdvancedController.h @@ -5,11 +5,12 @@ It may be used under the terms of the GNU General Public License. */ #import <Cocoa/Cocoa.h> +#import "HBViewValidation.h" /** * HBAdvancedController */ -@interface HBAdvancedController : NSViewController +@interface HBAdvancedController : NSViewController <HBViewValidation> { /* Advanced Tab for opts fX264optView*/ IBOutlet NSBox * fOptionsBox; @@ -69,7 +70,6 @@ - (NSString *) optionsStringLavc; - (void) setOptions: (NSString *)string; - (void) setLavcOptions: (NSString *)string; -- (void) enableUI: (bool) b; - (void) setHidden: (BOOL) hide; - (void) setLavcOptsEnabled: (BOOL) lavc; - (IBAction) X264AdvancedOptionsAnimate: (id) sender; diff --git a/macosx/HBAdvancedController.m b/macosx/HBAdvancedController.m index 0900c010e..e0f509235 100644 --- a/macosx/HBAdvancedController.m +++ b/macosx/HBAdvancedController.m @@ -76,7 +76,7 @@ return; } - - (void) enableUI: (bool) b + - (void)setUIEnabled:(BOOL)flag { unsigned i; NSControl * controls[] = @@ -97,16 +97,16 @@ NSTextField * tf = (NSTextField *) controls[i]; if( ![tf isBezeled] ) { - [tf setTextColor: b ? [NSColor controlTextColor] : + [tf setTextColor: flag ? [NSColor controlTextColor] : [NSColor disabledControlTextColor]]; continue; } } - [controls[i] setEnabled: b]; + [controls[i] setEnabled: flag]; } - [fX264optView setWantsLayer:YES]; + //[fX264optView setWantsLayer:YES]; } - (void)dealloc diff --git a/macosx/HBAudioController.h b/macosx/HBAudioController.h index 8655a0792..b77b3ad20 100644 --- a/macosx/HBAudioController.h +++ b/macosx/HBAudioController.h @@ -6,6 +6,7 @@ // #import <Cocoa/Cocoa.h> +#import "HBViewValidation.h" extern NSString *keyAudioTrackIndex; extern NSString *keyAudioTrackName; @@ -24,7 +25,7 @@ extern NSString *HBMixdownChangedNotification; * * Responds to HBContainerChangedNotification and HBTitleChangedNotification notifications. */ -@interface HBAudioController : NSViewController +@interface HBAudioController : NSViewController <HBViewValidation> @property (nonatomic, readonly, retain) NSArray *masterTrackArray; @property (nonatomic, readonly) NSDictionary *noneTrack; @@ -34,8 +35,6 @@ extern NSString *HBMixdownChangedNotification; // Get the list of audio tracks @property (readonly, nonatomic, copy) NSArray *audioTracks; -- (void) enableUI: (BOOL) b; - - (void) applySettingsFromPreset:(NSDictionary *)preset; - (void) addTracksFromQueue: (NSMutableDictionary *) aQueue; diff --git a/macosx/HBAudioController.m b/macosx/HBAudioController.m index 171f37e39..df623d877 100644 --- a/macosx/HBAudioController.m +++ b/macosx/HBAudioController.m @@ -95,13 +95,13 @@ NSString *HBMixdownChangedNotification = @"HBMixdownChangedNotification"; [self switchingTrackFromNone:nil]; } -- (void)enableUI:(BOOL)b +- (void)setUIEnabled:(BOOL)flag { - self.enabled = b; - [fTableView setEnabled:b]; - [self.trackPopup setEnabled:b]; - [self.configureDefaults setEnabled:b]; - [self.reloadDefaults setEnabled:b]; + self.enabled = flag; + [fTableView setEnabled:flag]; + [self.trackPopup setEnabled:flag]; + [self.configureDefaults setEnabled:flag]; + [self.reloadDefaults setEnabled:flag]; } - (BOOL)validateUserInterfaceItem:(id < NSValidatedUserInterfaceItem >)anItem diff --git a/macosx/HBChapterTitlesController.h b/macosx/HBChapterTitlesController.h index f56fd367b..0e59ce902 100644 --- a/macosx/HBChapterTitlesController.h +++ b/macosx/HBChapterTitlesController.h @@ -4,15 +4,15 @@ Homepage: <http://handbrake.fr/>. It may be used under the terms of the GNU General Public License. */ -#include <Cocoa/Cocoa.h> +#import <Cocoa/Cocoa.h> +#import "HBViewValidation.h" /** * HBChapterTitlesController * Responds to HBTitleChangedNotification notifications. */ -@interface HBChapterTitlesController : NSViewController +@interface HBChapterTitlesController : NSViewController <HBViewValidation> -- (void)enableUI:(BOOL)b; - (void)addChaptersFromQueue:(NSMutableArray *)newChaptersArray; /** diff --git a/macosx/HBChapterTitlesController.m b/macosx/HBChapterTitlesController.m index c4f6a6583..6702a5d81 100644 --- a/macosx/HBChapterTitlesController.m +++ b/macosx/HBChapterTitlesController.m @@ -83,12 +83,12 @@ [fChapterTable reloadData]; } -- (void)enableUI:(BOOL)b +- (void)setUIEnabled:(BOOL)flag { - [fCreateChaptersMarkers setEnabled:b]; - [fChapterTable setEnabled:b]; - [fLoadChaptersButton setEnabled:b]; - [fSaveChaptersButton setEnabled:b]; + [fCreateChaptersMarkers setEnabled:flag]; + [fChapterTable setEnabled:flag]; + [fLoadChaptersButton setEnabled:flag]; + [fSaveChaptersButton setEnabled:flag]; } - (NSArray *)chapterTitlesArray diff --git a/macosx/HBPresetsViewController.m b/macosx/HBPresetsViewController.m index 6dd76793a..08e03e31e 100644 --- a/macosx/HBPresetsViewController.m +++ b/macosx/HBPresetsViewController.m @@ -85,9 +85,9 @@ #pragma mark - HBViewValidation methods -- (void)setEnabled:(BOOL)enabled +- (void)setUIEnabled:(BOOL)flag { - _enabled = enabled; + self.enabled = flag; } #pragma mark - UI Methods @@ -135,18 +135,14 @@ - (IBAction)insertFolder:(id)sender { NSIndexPath *selectionIndexPath = [self.treeController selectionIndexPath]; - if (!selectionIndexPath) + if (!selectionIndexPath || [[[self.treeController selectedObjects] firstObject] isBuiltIn]) { selectionIndexPath = [NSIndexPath indexPathWithIndex:self.presets.root.children.count]; } - HBPreset *selectedNode = [[self.treeController selectedObjects] firstObject]; - if (!selectedNode.isBuiltIn) - { - HBPreset *node = [[HBPreset alloc] initWithFolderName:@"New Folder" builtIn:NO]; - [self.treeController insertObject:node atArrangedObjectIndexPath:selectionIndexPath]; - [node autorelease]; - } + HBPreset *node = [[HBPreset alloc] initWithFolderName:@"New Folder" builtIn:NO]; + [self.treeController insertObject:node atArrangedObjectIndexPath:selectionIndexPath]; + [node autorelease]; } - (IBAction)setDefault:(id)sender diff --git a/macosx/HBSubtitlesController.h b/macosx/HBSubtitlesController.h index 71bafe868..a117cee0f 100644 --- a/macosx/HBSubtitlesController.h +++ b/macosx/HBSubtitlesController.h @@ -5,6 +5,7 @@ It may be used under the terms of the GNU General Public License. */ #import <Cocoa/Cocoa.h> +#import "HBViewValidation.h" extern NSString *keySubTrackName; extern NSString *keySubTrackIndex; @@ -24,9 +25,8 @@ extern NSString *keySubTrackSrtCharCode; * HBSubtitlesController * Responds to HBContainerChangedNotification and HBTitleChangedNotification notifications. */ -@interface HBSubtitlesController : NSViewController +@interface HBSubtitlesController : NSViewController <HBViewValidation> -- (void)enableUI:(BOOL)b; - (void)addTracksFromQueue:(NSMutableArray *)newSubtitleArray; - (void)applySettingsFromPreset:(NSDictionary *)preset; diff --git a/macosx/HBSubtitlesController.m b/macosx/HBSubtitlesController.m index f9706973e..996105796 100644 --- a/macosx/HBSubtitlesController.m +++ b/macosx/HBSubtitlesController.m @@ -92,13 +92,13 @@ NSString *keySubTrackLanguageIndex = @"keySubTrackLanguageIndex"; return self; } -- (void)enableUI:(BOOL)b +- (void)setUIEnabled:(BOOL)flag { - [self.trackPopUp setEnabled:b]; - [self.configureDefaults setEnabled:b]; - [self.reloadDefaults setEnabled:b]; - [self.fTableView setEnabled:b]; - self.enabled = b; + [self.trackPopUp setEnabled:flag]; + [self.configureDefaults setEnabled:flag]; + [self.reloadDefaults setEnabled:flag]; + [self.fTableView setEnabled:flag]; + self.enabled = flag; } - (void)titleChanged:(NSNotification *)aNotification diff --git a/macosx/HBVideoController.h b/macosx/HBVideoController.h index 75a65f292..b3d0637bc 100644 --- a/macosx/HBVideoController.h +++ b/macosx/HBVideoController.h @@ -5,6 +5,8 @@ It may be used under the terms of the GNU General Public License. */ #import <Cocoa/Cocoa.h> +#import "HBViewValidation.h" + #include "hb.h" @class HBController; @@ -17,9 +19,7 @@ extern NSString *HBVideoEncoderChangedNotification; * * Responds to HBContainerChangedNotification and HBTitleChangedNotification notifications. */ -@interface HBVideoController : NSViewController - -- (void)enableUI:(BOOL)b; +@interface HBVideoController : NSViewController <HBViewValidation> // Methods to apply the settings to the controller - (void)applyVideoSettingsFromQueue:(NSDictionary *)queueToApply; diff --git a/macosx/HBVideoController.m b/macosx/HBVideoController.m index e602e3f8d..3490b483d 100644 --- a/macosx/HBVideoController.m +++ b/macosx/HBVideoController.m @@ -120,8 +120,8 @@ NSString *HBVideoEncoderChangedNotification = @"HBVideoEncoderChangedNotificatio * avoids an invalid free */ fX264PresetsUnparsedUTF8String = NULL; - _pictureFiltersField = @"Pictures Filters:"; - _pictureSettingsField = @"Pictures Settings:"; + _pictureFiltersField = @"Picture Filters:"; + _pictureSettingsField = @"Picture Settings:"; NSNotificationCenter *center = [NSNotificationCenter defaultCenter]; @@ -187,7 +187,7 @@ NSString *HBVideoEncoderChangedNotification = @"HBVideoEncoderChangedNotificatio [fVidTurboPassCheck setState: NSOffState]; } -- (void)enableUI:(BOOL)b { +- (void)setUIEnabled:(BOOL)flag { NSControl *controls[] = { fFramerateMatrix, @@ -214,17 +214,17 @@ NSString *HBVideoEncoderChangedNotification = @"HBVideoEncoderChangedNotificatio NSTextField *tf = (NSTextField *)controls[i]; if (![tf isBezeled]) { - [tf setTextColor: (b ? + [tf setTextColor: (flag ? [NSColor controlTextColor] : [NSColor disabledControlTextColor])]; continue; } } - [controls[i] setEnabled: b]; + [controls[i] setEnabled:flag]; } [self videoMatrixChanged:nil]; - [self enableX264Widgets:b]; + [self enableX264Widgets:flag]; } - (void)containerChanged:(NSNotification *)aNotification @@ -1065,7 +1065,7 @@ NSString *HBVideoEncoderChangedNotification = @"HBVideoEncoderChangedNotificatio // enable/disable the checkbox and advanced panel [fX264UseAdvancedOptionsCheck setEnabled:enable]; - [self.fAdvancedOptions enableUI:enable_advanced_panel]; + [self.fAdvancedOptions setUIEnabled:enable_advanced_panel]; // enable/disable the x264 preset system controls for (unsigned i = 0; i < (sizeof(controls) / sizeof(NSControl*)); i++) diff --git a/macosx/HBViewValidation.h b/macosx/HBViewValidation.h index 610d7bb25..779208cd4 100644 --- a/macosx/HBViewValidation.h +++ b/macosx/HBViewValidation.h @@ -10,6 +10,6 @@ @protocol HBViewValidation <NSObject> -- (void)setEnabled:(BOOL)enabled; +- (void)setUIEnabled:(BOOL)flag; @end |