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

Ctrl+drag builds last bridge type (if possible) #2238

Closed
DorpsGek opened this issue Aug 19, 2008 · 5 comments
Closed

Ctrl+drag builds last bridge type (if possible) #2238

DorpsGek opened this issue Aug 19, 2008 · 5 comments
Labels
component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay

Comments

@DorpsGek
Copy link
Member

planetmaker opened the ticket and wrote:

A frequent feature request is to build the same bridge type as built the last time a bridge was built. This patch introduces a global var which stores the last bridge type. Ctrl-drag bridge checks whether the length allows that stored type. If so, that type of bridge is built, otherwise the usual bridge selection dialogue is shown.

Patch for r14103 attached - I hope I followed style :)

Criticism and comments very much appreciated.

Attachments

Reported version: trunk
Operating system: All


This issue was imported from FlySpray: https://bugs.openttd.org/task/2238
@DorpsGek
Copy link
Member Author

sulai wrote:

I think this is the best solution. Even better than building the fastest one using ctrl. Thanks for the good work!


This comment was imported from FlySpray: https://bugs.openttd.org/task/2238#comment4775

@DorpsGek
Copy link
Member Author

DorpsGek commented Oct 2, 2008

planetmaker wrote:

update to trunk r14423.

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/2238#comment4807

@DorpsGek
Copy link
Member Author

Rubidium wrote:

Wouldn't it make sense to split the last built road bridge and last built rail bridge?


This comment was imported from FlySpray: https://bugs.openttd.org/task/2238#comment4957

@DorpsGek
Copy link
Member Author

planetmaker wrote:

Your wish is my command ;) - attached a diff to r14621.
So far I didn't have that desire but play styles are different. I introduced a 2nd global variable (is there a better way to do that?). The choice of switch instead of using an expression using ? allows easier code adaption should at one time in the future viaducts get their own different bridge types.

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/2238#comment5068

@DorpsGek
Copy link
Member Author

DorpsGek commented Jan 3, 2009

Rubidium closed the ticket.

Reason for closing: Implemented

In r14805


This comment was imported from FlySpray: https://bugs.openttd.org/task/2238

@DorpsGek DorpsGek closed this as completed Jan 3, 2009
@DorpsGek DorpsGek added component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay labels Apr 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay
Projects
None yet
Development

No branches or pull requests

1 participant