mirror of https://github.com/ppy/osu
fix selecting different tab causing filter to change by itself
and minor cleanup
This commit is contained in:
parent
f00676e6ee
commit
e2a1b18a2c
|
@ -64,12 +64,7 @@ public SocialOverlay()
|
|||
// TODO sort our list in some way (either locally or with API call)
|
||||
//Filter.DisplayStyleControl.Dropdown.Current.ValueChanged += rankStatus => Scheduler.AddOnce(updateSearch);
|
||||
|
||||
Header.Tabs.Current.ValueChanged += tab =>
|
||||
{
|
||||
currentQuery.Value = string.Empty;
|
||||
Filter.Tabs.Current.Value = (SocialSortCriteria)Header.Tabs.Current.Value;
|
||||
Scheduler.AddOnce(updateSearch);
|
||||
};
|
||||
Header.Tabs.Current.ValueChanged += _ => Scheduler.AddOnce(updateSearch);
|
||||
|
||||
currentQuery.ValueChanged += v =>
|
||||
{
|
||||
|
@ -86,7 +81,7 @@ public SocialOverlay()
|
|||
|
||||
currentQuery.BindTo(Filter.Search.Current);
|
||||
|
||||
Filter.Tabs.Current.ValueChanged += sortCriteria => Scheduler.AddOnce(updateSearch);
|
||||
Filter.Tabs.Current.ValueChanged += _ => Scheduler.AddOnce(updateSearch);
|
||||
|
||||
Scheduler.AddOnce(updateSearch); // so it displays something once it's first opened
|
||||
}
|
||||
|
@ -99,9 +94,6 @@ private void load(APIAccess api)
|
|||
|
||||
private void recreatePanels(PanelDisplayStyle displayStyle)
|
||||
{
|
||||
if (Users == null)
|
||||
return;
|
||||
|
||||
if (panels != null)
|
||||
{
|
||||
panels.FadeOut(200);
|
||||
|
@ -109,6 +101,9 @@ private void recreatePanels(PanelDisplayStyle displayStyle)
|
|||
panels = null;
|
||||
}
|
||||
|
||||
if (Users == null)
|
||||
return;
|
||||
|
||||
var newPanels = new FillFlowContainer<UserPanel>
|
||||
{
|
||||
RelativeSizeAxes = Axes.X,
|
||||
|
|
Loading…
Reference in New Issue