Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Square function bug on areas #682

Closed
Mapiate opened this issue Dec 1, 2022 · 5 comments
Closed

Square function bug on areas #682

Mapiate opened this issue Dec 1, 2022 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@Mapiate
Copy link

Mapiate commented Dec 1, 2022

Description

Using the Square function on areas either by right clicking or using the hotkey Q does not work as expected. There seems to be a delay and does not render until additional actions take place, like adding or deleting nodes, or moving the area. Trying to use the Square option again, is then disabled.

New Building where the Square function was used but did not show the new shape.
image

When using the Move function, the new shape then displays but leaves behind traces of the nodes.
image

I was able to reproduce this consistently in Firefox and Chrome.

Version

2.00-alpha.3

What browser are you seeing the problem on? What version are you running?

Firefox v107.0

Steps to reproduce

Create a new building.
Right click or use the Q hotkey to use the Square.

The browser URL at the time you encountered the bug

https://mapwith.ai/rapid-v2-alpha3#background=MAPNIK&datasets=fbRoads,msBuildings&disable_features=points,indoor,landuse,boundaries,rail,pistes,aerialways,past_future&id=w-5&map=19.00/43.41944/-71.95998

@bhousel
Copy link
Contributor

bhousel commented Dec 1, 2022

Thanks! There is a tricky bug that I'm working on this week where sometimes geometries get cached and not redrawn properly to reflect how they should look after edits happen. I've noticed this with circularizing too, and also across undoing-redoing edits. Hope to have a fix for this pretty soon.

@bhousel bhousel added alpha bug Something isn't working labels Dec 1, 2022
@Bonkles Bonkles added this to the v2 General Beta (Extended) milestone Dec 1, 2022
@Bonkles
Copy link
Contributor

Bonkles commented Dec 1, 2022

Thanks for the report! I'll add a little context here- after squaring the building the first time, RapiD's in-memory representation of the building has been updated to be square. When you right-click again, the square option isn't available because RapiD thinks the shape is square.

This causes understandable user confusion, because RapiD hasn't redrawn the shape to look square.

If you were to hit the browser reload button and keep your changes, you would see the squared building get rendered correctly.

UPDATE: It looks like this bears further scrutiny. Even after squaring a building and refreshing rapid / keeping changes, the unsquare building is still displayed. ?!?!?!?!

@cbeddow
Copy link
Contributor

cbeddow commented Dec 6, 2022

@Bonkles not sure if related, but when I draw something new even, a swimming pool here, I cannot square it (edit: refresh shows the squared one though after I submit the change): https://mapwith.ai/rapid-v2-alpha3#background=Bing&datasets=fbRoads,msBuildings&disable_features=boundaries,past_future,others&map=21.18/25.76330/-80.13800

@Bonkles
Copy link
Contributor

Bonkles commented Dec 6, 2022

After a little testing on my own this morning, it does indeed look like there's an issue with building squaring beyond the 'we didn't redraw it correctly'. I will dig in more for sure!

@Bonkles
Copy link
Contributor

Bonkles commented Dec 8, 2022

Update to the update- nope, square has the same bug that circularize, rotate, etc. all have right now. That is, until the fix we just pushed!

Fixed in alpha 3.2 release that just went live.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: 🏁 Done
Development

No branches or pull requests

4 participants