Modify ↓

Opened 3 years ago

Closed 3 years ago

#5560 closed defect (fixed)

[PATCH] Extrude does not word with multiple adjacent buildings

Reported by: bastiK Owned by: Ldp
Priority: normal Component: Core
Version: latest Keywords:
Cc:

Description

When you have terraced buildings and try to extrude a front, the other buildings get distorted.

Attachments (4)

extrude-buildings.png​ (31.9 KB) - added by bastiK 3 years ago.
patch.diff​ (8.1 KB) - added by extropy 3 years ago.
extrude-buildings.osm​ (1.0 KB) - added by bastiK 3 years ago.
extrudeBuilgins2.diff​ (995 bytes) - added by extropy 3 years ago.
Fixed a bug with buildings.

Download all attachments as: .zip

Change History (9)

Changed 3 years ago by bastiK

comment:1 Changed 3 years ago by extropy

Working on this.

Changed 3 years ago by extropy

comment:2 Changed 3 years ago by extropy

  • Summary changed from Extrude does not word with multiple adjacent buildings to [PATCH] Extrude does not word with multiple adjacent buildings

Added a patch that fixes this.
Please review and commit.

Additionally improved the geometry functions a bit.

comment:3 Changed 3 years ago by bastiK

In [3631/josm]:

see #5560 (patch by extropy) - Extrude does not work with multiple adjacent buildings

comment:4 Changed 3 years ago by bastiK

Does not work for all cases: Try to extrude the lower segments of the example:

Changed 3 years ago by bastiK

Changed 3 years ago by extropy

Fixed a bug with buildings.

comment:5 Changed 3 years ago by bastiK

  • Resolution set to fixed
  • Status changed from new to closed

In [3634/josm]:

applied #5560 (patch by extropy) - Extrude does not work with multiple adjacent buildings

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed .
as The resolution will be set. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.