Click and drag for way objects May 13, 2009, 01:08:50 pm Since the incorporation of the feature allowing players to build ways by clicking and dragging, rather clicking once on the start point, and again on the end point, building ways has become much easier, more pleasant, and more user-friendly.Would there be any possibility of extending the same system to way objects, such as electrification? Having one system (click and drag) for laying ways, and a different system (click twice) for laying way objects is confusing, and the click and drag method (with the estimated building costs) is much more helpful and easier to use. Quote Selected
Re: Click and drag for way objects Reply #1 – May 13, 2009, 01:21:37 pm I support this request. But at least for power rails, it is necessary to extend zeiger_t by get_after_bild. Or is there an other cl****, which can be used instead of zeiger_t? Quote Selected
Re: Click and drag for way objects Reply #2 – May 13, 2009, 01:30:51 pm This extension would be not too m****ive, I guess Quote Selected
Re: Click and drag for way objects Reply #3 – May 14, 2009, 08:53:54 am Any ding has a get after bild. Extending zeiger is only logical for OOP. Quote Selected
Re: Click and drag for way objects Reply #4 – May 14, 2009, 09:01:27 am Quote from: prissi – on May 14, 2009, 08:53:54 amAny ding has a get after bild. Extending zeiger is only logical for OOP.Yes, of course. I only wanted to say, that you have define the (virtual) function get_after_bild for zeiger_t, since it isn't defined yet. Quote Selected
Re: Click and drag for way objects Reply #5 – May 22, 2009, 06:39:27 am You can find a patch at http://simutrans-forum.de/forum/thread.php?postid=50586#post50586. Quote Selected
Re: Click and drag for way objects Reply #6 – May 22, 2009, 02:45:05 pm Gerw,excellent work! I have tested it and it works marvellously. Thank you!Edit: Now implemented into Simutrans-Experimental version 3.12. Thank you, Gerw! Quote Selected Last Edit: May 23, 2009, 10:22:35 pm by jamespetts
Re: Click and drag for way objects Reply #7 – May 24, 2009, 11:55:50 am Thank you, JamesPetts.Here is a new version with new (internal) features:- a new cl**** two_click_werkzeug_t for tools, which needs two clicks. It also allows dragging. In the current version way and way object building are implemented as two click tools. But now, other tools can also easily support dragging.- a new cl**** koord3d_vector with two little helper functions.- way and way object building now allows rotating between the two clicks (this is buggy in current versions).You can find the patch at www.tu-chemnitz.de/~gerw/drag_wayobj_v2.patch.Can someone move this thread to 'patches'? Quote Selected
Re: Click and drag for way objects Reply #8 – May 24, 2009, 01:11:10 pm One problem.1. Choose way tool2. Click empty land3. Click on a building4. Click another empty landResult: Way was build from first clicked empty land to last clicked empty land.Note: When I clicked on a building, zeiger on the first clicked land will be removed. But tool itself is not canceled. This should be canceled. This problem will happen with dragging, too. And also wayobject tools have similar problem. Quote Selected
Re: Click and drag for way objects Reply #9 – May 24, 2009, 01:24:17 pm Thank you for testing.It's now fixed (only one line...) Quote Selected
Re: Click and drag for way objects Reply #10 – May 24, 2009, 02:03:14 pm Thank you. Another small thing about estimate tooltips.- Way tools show positive value but wayobject tools show negative value.- Upgrade wayobject doesn't show correct estimate value on tooltips. If I drag on lower wayobject, it shows zero. But if this estimate would be difficult, it's Okay. Quote Selected
Re: Click and drag for way objects Reply #11 – May 24, 2009, 02:12:12 pm Thank you again. Both fixed.See www.tu-chemnitz.de/~gerw/drag_wayobj_v3.patch Quote Selected
Re: Click and drag for way objects Reply #12 – May 26, 2009, 04:55:59 pm Now also the forest and the wayremover tool are implemented as two_click_tools.www.tu-chemnitz.de/~gerw/drag_wayobj_v4.patch Quote Selected
Re: Click and drag for way objects Reply #13 – May 26, 2009, 06:52:01 pm Gerw,what do you mean by two click tools? Quote Selected
Re: Click and drag for way objects Reply #14 – May 26, 2009, 06:58:24 pm Tools that use click-and-drag ... ie tools that give you a preview of the result (as the original waybuilder). Quote Selected
Re: Click and drag for way objects Reply #15 – May 27, 2009, 06:32:06 am Quote from: Dwachs – on May 26, 2009, 06:58:24 pmTools that use click-and-drag ... ie tools that give you a preview of the result (as the original waybuilder).That's not the whole truth Two click tools are tools that need two koord3d in order to work. I've implemented a new cl****, which directly support this. The implementation of the tools itself only need to declare two methods, one that provide a preview (more or less optional) and one method, which does the real work. In the trunk version, the code in some tools (way builder, way remover, forest builder) was doubled, so I decided to derive a tool cl****, which does this work. Quote Selected
Re: Click and drag for way objects Reply #16 – May 27, 2009, 02:25:03 pm One small problem.If I drag to outside of the map (outside sea) and mouse up, tool works.But when I move back cursor to inside of the map, a new zeiger will appear at end position. Quote Selected
Re: Click and drag for way objects Reply #17 – May 27, 2009, 04:15:41 pm Thank you for your "little problems" www.tu-chemnitz.de/~gerw/drag_wayobj_v5.patch Quote Selected
Re: Click and drag for way objects Reply #18 – May 27, 2009, 04:54:30 pm Solved. Thank you. Quote Selected
Re: Click and drag for way objects Reply #19 – May 27, 2009, 08:10:48 pm Since there are more koord list, I would suggest to add the koord_vector and koord3d_vector to koord.h/cc and koord3d.h/cc as the logical places for them. Defining a new file with a single most trivial derived cl**** for use in almost only one place seems too much. That way also the factor kood lists (and other too) could be unified; this would also make the code much cleaner. Quote Selected
Re: Click and drag for way objects Reply #20 – May 28, 2009, 07:54:58 am Is there any drawback when putting this cl****es in an extra file? So I would prefer an extra file for this cl****, since there would be more members in this cl**** in future and therefore it's more clear in an extra file. Quote Selected
Re: Click and drag for way objects Reply #21 – May 28, 2009, 01:27:52 pm The cl**** is derived from koord3d. It is used currently at one position. Making a new file for this purpose imho adds more confusion and clutter than adding it to koord3d directly. (Since the rotation is defined there anyway.)You do not put any C function is a seperate file, if it is just a printf and an addition, do you?On a side note, rather hang_t should get its own file, but is currently a member of koord ... Quote Selected
Re: Click and drag for way objects Reply #22 – May 28, 2009, 04:02:45 pm Don't let us argue about such small things. I've now put it in kood3d.h.With this new patch, the wayobjects get connected automatically.www.tu-chemnitz.de/~gerw/drag_wayobj_v6.patchBTW: What sense have the wkz->move messages with buttonstate==0? They are only called after dragging, when releasing the mouse button (but not when you release the mousebutton outside the map - as z9999 showed).What about sending wkz->move also when no button is pressed? Then we could have a way builder preview even without dragging. Of course, this would have any drawbacks, if you want to build a way over the whole map.If tested it and it works (at least for way builder/remover, wayobj builder):www.tu-chemnitz.de/~gerw/drag_wayobj_v6_advanced_move.patch Quote Selected
Re: Click and drag for way objects Reply #23 – May 28, 2009, 09:08:31 pm The waybuilder without dragging is very consuming on the CPU and will put Netbooks into big troubles. Especially when no route is possible. Dragging needs to be optional.wkz->move was once intended for more. Do to bugs and errors in the implementation it ended like this. Before, there was also no call to wkz->work. Quote Selected
Re: Click and drag for way objects Reply #24 – May 29, 2009, 06:30:41 am Quote from: prissi – on May 28, 2009, 09:08:31 pmThe waybuilder without dragging is very consuming on the CPU and will put Netbooks into big troubles. Especially when no route is possible. Dragging needs to be optional.Yes, of course, this is true. But in my opinion, it is more comfortable to build ways without dragging and with preview. I've tested it yesterday and it was some kind of easy I think, making it configurable would be a good idea, but will also blow up the config...Quote from: prissi – on May 28, 2009, 09:08:31 pmwkz->move was once intended for more. Do to bugs and errors in the implementation it ended like this. Before, there was also no call to wkz->work.What was the intention of wkz->move? Should we simply remove this call with buttonstate==0 (all tools ignore this) or is there an idea, which could be completed? Quote Selected
Re: Click and drag for way objects Reply #25 – May 29, 2009, 08:17:39 am As reported in other problem report, in Windows GDI version, there is a performance problem with CRTL key on pause mode. Quote Selected
Re: Click and drag for way objects Reply #26 – May 29, 2009, 10:19:19 am The call with buttonstate==0 was done for moving without dragging. I would suggest to keep it for now. Quote Selected
Re: Click and drag for way objects Reply #27 – May 29, 2009, 11:58:34 am Quote from: prissi – on May 29, 2009, 10:19:19 amThe call with buttonstate==0 was done for moving without dragging. I would suggest to keep it for now.Like in the advanced move patch above?Quote from: z9999 – on May 29, 2009, 08:17:39 amAs reported in other problem report, in Windows GDI version, there is a performance problem with CRTL key on pause mode.Only with this patch? And which version of this patch? Quote Selected
Re: Click and drag for way objects Reply #28 – May 29, 2009, 06:05:49 pm New version of the patch. Now you can also delete way objects with dragging. It's included in the wkz_wayobj_t, since it uses almost the same code. But of course, it's sometimes hard to read.Lines for your menutab.conf:Code: [Select]general_tool[33]=,1,11,toolbar[2][11]=general_tool[33],16,,2www.tu-chemnitz.de/~gerw/drag_wayobj_v7.patch Quote Selected
Topic: Click and drag for way objects (Read 9437 times)
previous topic - next topic
