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.

Sign up for GitHub

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

Jump to bottom

Cannot place the Egyptian pyramid in the air #10477

Closed
Gymnasiast opened this issue Dec 30, 2019 · 3 comments
Closed

Cannot place the Egyptian pyramid in the air #10477

Gymnasiast opened this issue Dec 30, 2019 · 3 comments
Labels
bug Something went wrong.

Comments

@Gymnasiast
Copy link
Member

OS: Ubuntu 19.10
Version: 0.2.4+
Commit/Build: 4b0dff9

Once you start holding shift, the preview will disappear and attempting to place it will only yield Cannot position this here... with no further explanation.

Tagging @duncanspumpkin and @tupaschoal .
@Gymnasiast Gymnasiast added the regression An issue that was made since the last release. label Dec 30, 2019
@tupaschoal
Copy link
Member

tupaschoal commented Dec 30, 2019 •
edited
Loading

Weirdly, it triggers a failed assertion on debug builds earlier than January. I will bisect with production to see if it is also that old.

@Gymnasiast Gymnasiast added bug Something went wrong. and removed regression An issue that was made since the last release. labels Dec 30, 2019
@duncanspumpkin
Copy link
Contributor

Yeah i know the source of this bug. It's to do with window scaling and protections implemented with that. If you look back a few commits i said this isn't correct but will prevent a crash.

@duncanspumpkin
Copy link
Contributor

duncanspumpkin commented Dec 30, 2019 •
edited
Loading

#10405 is the PR where I fixed the crash that would happen. Also see my comment about how this behaviour differs from vanilla.

#7868 is the ultimate source of this bug.

duncanspumpkin added a commit to duncanspumpkin/OpenRCT2 that referenced this issue Dec 30, 2019
@duncanspumpkin
Fix OpenRCT2#10477. Fix upper max height limit being very low for lar…
ec96be0 
…ge scenery

Mistake made while improving mouse control when zoomed out caused the upper limit for multi tile scenery to be limited by the sum of all clearance z values instead of the greatest clearance z value.
duncanspumpkin added a commit to duncanspumpkin/OpenRCT2 that referenced this issue Dec 31, 2019
@duncanspumpkin
Fix OpenRCT2#10477. Fix upper max height limit being very low for lar…
970ac45 
…ge scenery

Mistake made while improving mouse control when zoomed out caused the upper limit for multi tile scenery to be limited by the sum of all clearance z values instead of the greatest clearance z value.
janisozaur added a commit that referenced this issue Mar 24, 2020
@janisozaur
Release 0.2.5
4f6e77e 
- Feature: [#3154] Use a random title sequence each time it is shown.
- Feature: [#6553] Android version now runs in full screen.
- Feature: [#7865] Transport rides can now be synchronised.
- Feature: [#9073] Shortcut keys for the Tile Inspector.
- Feature: [#10305] Add two shortcuts for increasing and decreasing the scaling factor.
- Feature: [#10189] Make Track Designs work in multiplayer.
- Feature: [#10357] Added window for scenery scatter tool, allowing for area and density selection.
- Feature: [#10637] Console command to remove all floating objects.
- Change: [#1164] Use available translations for shortcut key bindings.
- Change: [#10997] Speed is automatically reset to normal upon scenario completion.
- Fix: [#2485] Hide Vertical Faces not applied to the edges of water.
- Fix: [#5249] No collision detection when building ride entrance at heights > 85.5m.
- Fix: [#6766] Changelog window doesn't open on some platforms.
- Fix: [#7784] Vehicle tab takes 1st car colour instead of tab_vehicle's colour.
- Fix: [#7854] Cannot build a custom spiral roller coaster design.
- Fix: [#7854] Empty entries in spiral roller coaster designs list.
- Fix: [#8151] Game freezes upon demolishing mazes at odd heights.
- Fix: [#8875] RCT1 competition scenarios are classified incorrectly.
- Fix: [#10176] Mistake in the sprite for the land tool's 6x6 grid.
- Fix: [#10196] Doors unable to be placed at end of track corners.
- Fix: [#10228] Can't import RCT1 Deluxe from Steam.
- Fix: [#10313] Path furniture can be placed on level crossings.
- Fix: [#10325] Crash when banners have no text.
- Fix: [#10376] No ratings generated when a shop and track intersect.
- Fix: [#10420] Money effect causing false positive desync.
- Fix: [#10477] Large Scenery cannot be placed higher using SHIFT.
- Fix: [#10489] Hosts last player action not being synchronized.
- Fix: [#10543] Secondary shop item prices are not imported correctly from RCT1 saves.
- Fix: [#10547] RCT1 parks have too many rides available.
- Fix: [#10587] Update last action coordinates on correct player.
- Fix: [#10631] Game bugs out and crashes if you get too many stations via copying stations with the tile inspector.
- Fix: [#10662] Duck cheat tooltips look odd and do not explain anything.
- Fix: [#10694] The lift hill speed of the flying roller coaster cannot be changed (original bug).
- Fix: [#10705] Apply multithreaded rendering to all viewports.
- Fix: [#10739] Mountain tool overlay for even-numbered selections.
- Fix: [#10752] Mute button state not correctly set at startup.
- Fix: [#10822] Can place too many peep spawns.
- Fix: [#10898] Banner text has an offset in tile inspector window.
- Fix: [#10904] RCT1/LL-scenarios with red water won't open.
- Fix: [#10941] The Clear Scenery tool gives refunds for ghost elements.
- Fix: [#10963] Light effects are drawn off-centre in some rotations.
- Fix: [#10993] Bottom toolbar not refreshing when a guest leaves the park.
- Fix: [#11001] Rides list does not use natural sorting.
- Fix: [objects#54] Stage Coach cars are not considered covered by the game.
- Fix: [objects#56] Handymen cut grass incorrectly.
- Improved: [#682] The staff patrol area is now drawn on the water, instead of on the surface under water.
- Improved: [#10858] Added horizontal grid lines to finance charts.
- Improved: [#10884] Added y-axes and labels to park window charts.
- Improved: [#10970] Introduced optional light effects for vehicles at night.
- Removed: [#6898] LOADMM and LOADRCT1 title sequence commands (use LOADSC instead).
XplosiveLugnut pushed a commit to XplosiveLugnut/OpenRCT2 that referenced this issue Apr 27, 2020
@duncanspumpkin
Fix OpenRCT2#10477. Fix upper max height limit being very low for lar…
69565a0 
…ge scenery

Mistake made while improving mouse control when zoomed out caused the upper limit for multi tile scenery to be limited by the sum of all clearance z values instead of the greatest clearance z value.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something went wrong.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@duncanspumpkin @Gymnasiast @tupaschoal