a-b-street/abstreet
1
1
Fork 0
mirror of https://github.com/a-b-street/abstreet.git synced 2025-01-04 20:44:52 +03:00
Code Issues Projects Releases Wiki Activity

Make the multi-road editor only check and edit lanes_ltr, not speed limit. The Aurora cycletrack edit stopped matching because the speed limit happens to change along Aurora. We can think through a better UX to handle both, but I suspect most use cases will focus on lane configuration. #597

Browse Source
This commit is contained in:
Dustin Carlino 2021-06-20 10:00:05 -07:00
parent e6e6fb267d
commit d8cdfaba05
1 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
game/src/edit/multiple_roads.rs
View File

@ -1,3 +1,6 @@
//! After a single road has been edited, these states let the changes be copied to all similar road
//! segments. Note that only lane configuration is copied, not speed limit or access restrictions.
use std::collections::HashSet;
use geom::Distance;
@ -34,13 +37,15 @@ impl SelectSegments {
new_state: EditRoad,
base_edits: MapEdits,
) -> Box<dyn State<App>> {
// Find all road segments matching the original state and name. base_road has already changed to new_state,
// so no need to exclude it.
// Find all road segments matching the original state and name. base_road has already
// changed to new_state, so no need to exclude it.
let map = &app.primary.map;
let base_name = map.get_r(base_road).get_name(None);
let mut candidates = HashSet::new();
for r in map.all_roads() {
if map.get_r_edit(r.id) == orig_state && r.get_name(None) == base_name {
if map.get_r_edit(r.id).lanes_ltr == orig_state.lanes_ltr
&& r.get_name(None) == base_name
{
candidates.insert(r.id);
}
}
@ -103,6 +108,7 @@ impl SelectSegments {
self.base_road.0
)
.text_widget(ctx),
// TODO Explain that this is only for lane configuration, NOT speed limit
Widget::row(vec![
"Click to select/unselect".text_widget(ctx).centered_vert(),
ctx.style()
@ -144,7 +150,7 @@ impl State<App> for SelectSegments {
edits
.commands
.push(app.primary.map.edit_road_cmd(*r, |new| {
*new = self.new_state.clone();
new.lanes_ltr = self.new_state.lanes_ltr.clone();
}));
}
apply_map_edits(ctx, app, edits);