Koda

Welcome to Koda bug tracker!

At the moment you can post bugreports or suggestions anonymously. But registered users have some benefits: attaching files, commenting, voting, tracking tasks e.t.c.

Due some asshole started spamming tracker, now (at least for some time) it’s not allowed to create anonymous tasks. Sorry for unconvenience.

In edit fields you can use DokuWiki syntax for formatting.

Tasklist

FS#182 - Tools | Generating Options | Template doesn't show Events Example

Attached to Project: Koda
Opened by Chris Haslam (c.haslam) - Sunday, 20 February 2011, 07:27 GMT+3
Type Bug Report
Category Application
Status Assigned
Assigned To Admin (Lazycat)
Operating System All
Severity Low
Priority Normal
Reported Version Development
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Votes 0
Private No

Details

But Events Example is in Code Templates dialog

This task depends upon

Comment by Admin (Lazycat) - Monday, 21 February 2011, 12:16 GMT+3

This combo showing only applicable templates. If you check "Generate OneEvent code", you will see it.

Comment by Chris Haslam (c.haslam) - Tuesday, 22 February 2011, 08:26 GMT+3

I understand, but ...

I think the screenshot below is clearer:

I can then say in the documentation that the templates available in the drop-down depend on the GUI Mode.

Note that, because MessageLoop mode is selected, Generate events for all controls is disabled. If the user chooses OnEvent mode, it would be enabled.

Even clearer would be to replace Generate events for all controls with

Generate events for:

  • Controls with OnClick specified
  • All controls

BTW My screenshot shows an empty group for Formatting. It is empty because I am lazy! It would contains the same controls as you have now.

Comment by Admin (Lazycat) - Tuesday, 22 February 2011, 13:59 GMT+3

Seems "Generate events for all controls" is not clear. This is generate not only all events in OnEvent mode, but also all "Case ControlN" variables in message loop mode. This probably should be "Generate variables/events for all controls".

This function was once added by request, but to be honest, I still think it's almost useless and unclear.

Comment by Chris Haslam (c.haslam) - Wednesday, 23 February 2011, 08:04 GMT+3
This probably should be "Generate variables/events for all controls"

If you really want to keep this as a single checkbox, how about "Generate cases/functions for all controls"?

What do you think of having the first choice in this dialog be 2 radios: Message loop or OnEvent?

Comment by Admin (Lazycat) - Wednesday, 23 February 2011, 19:29 GMT+3

Radios can be ok, but in this case it will be inconsistent with default options. Now those dialogs are intentionally the same.

"Generate cases/functions for all controls" is ok for me.

Comment by Chris Haslam (c.haslam) - Thursday, 24 February 2011, 07:15 GMT+3

By "default options" do you mean Options → Options → Code Generator? If so, I think that it would be clearer if the Generate OnEvent code checkbox were replaced with two radio buttons.

I think that one code-generation template could contain stuff for both Loop and OnEvent (as Default does now). Am I correct?

Comment by Admin (Lazycat) - Thursday, 24 February 2011, 11:33 GMT+3

I think your idea with radio buttons is right. This should make things clearer. I'm going to change both dialogs.

About templates - no, both modes can't be in one template. Default template is internally two different templates too.

Loading...