Skip to content

Commit

Permalink
iOS: Fixed bugs in NavigationManager after Android changes. Resetting…
Browse files Browse the repository at this point in the history
… scrollbar position to top when refreshing MobileLibraryBrowser.

Related to issue #405.
  • Loading branch information
ycastonguay committed Jul 4, 2013
1 parent e4d9734 commit 65643ed
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 2 deletions.
2 changes: 1 addition & 1 deletion MPfm/MPfm.MVP/Navigation/MobileNavigationManager.cs
Expand Up @@ -584,7 +584,7 @@ protected virtual void CreateSyncViewInternal(Action<IBaseView> onViewReady)
_syncView = Bootstrapper.GetContainer().Resolve<ISyncView>(new NamedParameterOverloads() { { "onViewReady", onViewReady } });

#if !ANDROID
PushTabView(MobileNavigationTabType.More, _equalizerPresetsView);
PushTabView(MobileNavigationTabType.More, _syncView);
#endif
}

Expand Down
Expand Up @@ -553,6 +553,9 @@ public void RefreshLibraryBrowser(IEnumerable<LibraryBrowserEntity> entities, Mo
_navigationBarTitle = navigationBarTitle;
_navigationBarSubtitle = navigationBarSubtitle;
// Reset scroll bar
tableView.ScrollRectToVisible(new RectangleF(0, 0, 1, 1), false);
if(browserType == MobileLibraryBrowserType.Albums)
{
tableView.Hidden = true;
Expand Down
4 changes: 3 additions & 1 deletion MPfm/MPfm.iOS/Classes/Navigation/iOSNavigationManager.cs
Expand Up @@ -48,7 +48,9 @@ public iOSNavigationManager(ITinyMessengerHub messageHub)
PushTabView(navCtrl.TabType, PlayerView);
break;
case MobileNavigationManagerCommandMessageType.ShowEqualizerPresetsView:
if(EqualizerPresetsView != null)
if(EqualizerPresetsView == null)
CreateEqualizerPresetsView(null);
else
PushDialogView("Equalizer Presets", null, EqualizerPresetsView);
break;
}
Expand Down

0 comments on commit 65643ed

Please sign in to comment.