a-b-street/abstreet
1
1
Fork 0
mirror of https://github.com/a-b-street/abstreet.git synced 2024-11-23 08:54:30 +03:00
Code Issues Projects Releases Wiki Activity

Make consultation mode and loading a proposal (locally or remote) work together correctly. [rebuild] [release]

Browse Source 
Well, work better. When you start a new proposal, a double entry will
get created in the proposal list.
This commit is contained in:
Dustin Carlino 2022-08-28 11:02:15 +01:00
parent d978029b11
commit fbdb5dc210
1 changed files with 9 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
apps/ltn/src/lib.rs
View File

@ -121,7 +121,8 @@ fn run(mut settings: Settings) {
.layers
.event(ctx, &app.cs, components::Mode::BrowseNeighbourhoods);
// Load a proposal first?
// Load a proposal first? Make sure to restore the partitioning from a file before
// calling BrowseNeighbourhoods
if let Some(ref name) = args.proposal {
// Remote edits require another intermediate state to load
if let Some(id) = name.strip_prefix("remote/") {
@ -167,10 +168,13 @@ fn setup_initial_states(
_ => panic!("Unknown Bristol consultation mode {consultation}"),
};
app.session.alt_proposals = crate::save::AltProposals::new();
ctx.loading_screen("initialize", |ctx, timer| {
crate::clear_current_proposal(ctx, app, timer);
});
// If we already loaded something from a saved proposal, then don't clear anything
if &app.session.partitioning.map != app.map.get_name() {
app.session.alt_proposals = crate::save::AltProposals::new();
ctx.loading_screen("initialize", |ctx, timer| {
crate::clear_current_proposal(ctx, app, timer);
});
}
// Look for the neighbourhood containing one small street
let r = app
@ -208,8 +212,6 @@ fn setup_initial_states(
));
states.push(BrowseNeighbourhoods::new_state(ctx, app));
}
// Restore the partitioning from a file before calling BrowseNeighbourhoods
// TODO Correct now? Comment OK? What about clear_current_proposal above?
if let Some(state) = popup_state {
states.push(state);
}