Sign in Join

Live chat for Free Pascal, Lazarus, Delphi and every Pascal programmer.

Mailing lists / lazarus / 2009-10

Bug (whis) at Procedure List

9 messages · 5 people · –

Reply by email Subscribe to this list Your subscription

Read-only archive. Use j and k to move between messages.

Raistware

Bug at Procedure List window:

Sorry for posting here that, but I didn't found a wiki page where to report that bug.
At "Procedure List window" should be selected "All" by default. By now it's selected 'None' and this is annoying, at GExperts, when the window is raised, it selected 'All objects' by default.
At least should be user configurable at Environment options..

Thanks Graeme for the plug-in.

__________ Informaci?n de ESET NOD32 Antivirus, versi?n de la base de firmas de virus 4475 (20091002) __________

ESET NOD32 Antivirus ha comprobado este mensaje.

http://www.eset.com

Reply by email

Graeme Geldenhuys
to Raistware

2009/10/2 Raistware :

At "Procedure List window" should be selected "All" by default. By now it's

You mean the combobox in the top right of the screen? I'm using v.0.9.28 and v.0.9.29 and in both versions of Lazarus, the "All" is the default option in the Object combobox.

What version of Lazarus are you using? Also what platform/ widgetset are you using?

Regards, - Graeme - _______________________________________________ fpGUI - a cross-platform Free Pascal GUI toolkit http://opensoft.homeip.net/fpgui/

Reply by email

Pierre

Hi all,

I also noticed this problem with 0.9.26 and with the 0.9.28.1 (downloaded 2 days ago). OS : WINXP

Thanks to all the lazarus developers.

Regards,

Pierre Delore http://datalinkwristapps.free.fr http://dpsite.free.fr ----- "Graeme Geldenhuys" a ?crit : > 2009/10/2 Raistware : > > At "Procedure List window" should be selected "All" by default. By > now it's > > You mean the combobox in the top right of the screen? I'm using > v.0.9.28 and v.0.9.29 and in both versions of Lazarus, the "All" is > the default option in the Object combobox. > > What version of Lazarus are you using? Also what platform/ widgetset > are you using? > > -- > Regards, > - Graeme - > > > _______________________________________________ > fpGUI - a cross-platform Free Pascal GUI toolkit > http://opensoft.homeip.net/fpgui/ > > -- > _______________________________________________ > Lazarus mailing list > > http://lists.lazarus.freepascal.org/mailman/listinfo/lazarus

Reply by email

Raul ..

Graeme Geldenhuys escribi?:

Show quoted text 10 lines
At "Procedure List window" should be selected "All" by default. By now it's You mean the combobox in the top right of the screen? I'm using v.0.9.28 and v.0.9.29 and in both versions of Lazarus, the "All" is the default option in the Object combobox. What version of Lazarus are you using? Also what platform/ widgetset are you using?

Yes, the top right combobox. I allways see it to 'None' ...

Lazarus last 0.9.29 git version (commit
1ae02800d1ee44fa59c56e77f0c0f591f8adf8bc) under Free Pascal Compiler version 2.3.1 [2009/06/08] for i386
Linux Debian. Gtk2

I reply only to Graeme, sorry

Reply by email

Raistware
to Graeme Geldenhuys: 2009/10/2 Raistware : You mean the combobox in the top right of the screen? I'm using…

Graeme Geldenhuys escribi?:

Show quoted text 13 lines
2009/10/2 Raistware : > At "Procedure List window" should be selected "All" by default. By now it's > You mean the combobox in the top right of the screen? I'm using v.0.9.28 and v.0.9.29 and in both versions of Lazarus, the "All" is the default option in the Object combobox. What version of Lazarus are you using? Also what platform/ widgetset are you using?

A little note, I have Lazarus in spanish, and I also use de Borland Directives to class notations (all classes start with T). At spanish:
"All" is translated as "Todo" and "None" is translated as "Ninguno"

So, In the listbox I have:
Ninguno
Tclass1
Tclass2
<Todo>

So it seem to be a problem with sorting, that you assumes that the first item should always be 'All' ?

Hope helps

__________ Informaci?n de ESET NOD32 Antivirus, versi?n de la base de firmas de virus 4512 (20091015) __________

ESET NOD32 Antivirus ha comprobado este mensaje.

http://www.eset.com

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lazarus.freepascal.org/pipermail/lazarus/attachments/20091016/afa5ac79/attachment.html>

Reply by email

Graeme Geldenhuys
to Raistware

2009/10/16 Raistware :

Show quoted text 9 lines
So, In the listbox I have: Ninguno Tclass1 Tclass2 <Todo> So it seem to be a problem with sorting, that you assumes that the first item should always be 'All' ?

Ah, that would explain it then. I'm not 100% sure about how to get around this though. Maybe sort the list with class names only, then disable sorting and then insert <all> and <none> in the top of the list. That could work? <all> (or whatever it's resource string value is), will then always be at the top.

Regards, - Graeme - _______________________________________________ fpGUI - a cross-platform Free Pascal GUI toolkit http://opensoft.homeip.net/fpgui/

Reply by email

Raistware
to Graeme Geldenhuys

Graeme Geldenhuys escribi?:

Show quoted text 22 lines
2009/10/16 Raistware : > So, In the listbox I have: > Ninguno > Tclass1 > Tclass2 > <Todo> > > > So it seem to be a problem with sorting, that you assumes that the first > item should always be 'All' ? > Ah, that would explain it then. I'm not 100% sure about how to get around this though. Maybe sort the list with class names only, then disable sorting and then insert <all> and <none> in the top of the list. That could work? <all> (or whatever it's resource string value is), will then always be at the top.

I think that should be a better solution to find the index of the '<all>' resourcestring and make it the selected option ... but I don't know if it's posible or if it would be easy to accomplish.

__________ Informaci?n de ESET NOD32 Antivirus, versi?n de la base de firmas de virus 4514 (20091016) __________

ESET NOD32 Antivirus ha comprobado este mensaje.

http://www.eset.com

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lazarus.freepascal.org/pipermail/lazarus/attachments/20091016/e1196ce7/attachment.html>

Reply by email

Alexander Klenin
to Graeme Geldenhuys: 2009/10/16 Raistware : Ah, that would explain it then. I'm not 100% sure about how to get…
Graeme Geldenhuys wrote 17 lines
2009/10/16 Raistware : > So, In the listbox I have: > Ninguno > Tclass1 > Tclass2 > <Todo> > > > So it seem to be a problem with sorting, that you assumes that the first > item should always be 'All' ? Ah, that would explain it then. I'm not 100% sure about how to get around this though. Maybe sort the list with class names only, then disable sorting and then insert <all> and <none> in the top of the list. That could work? <all> (or whatever it's resource string value is), will then always be at the top.

Alternatively, you can make these strings literally "<all>" and "<none>", including angle brackets.
This will also resolve potential confusion if the user will create classes with these names.

Alexander S. Klenin

Reply by email

Raistware
to Graeme Geldenhuys: 2009/10/16 Raistware : Ah, that would explain it then. I'm not 100% sure about how to get…

Graeme Geldenhuys escribi?:

Show quoted text 22 lines
2009/10/16 Raistware : > So, In the listbox I have: > Ninguno > Tclass1 > Tclass2 > <Todo> > > > So it seem to be a problem with sorting, that you assumes that the first > item should always be 'All' ? > Ah, that would explain it then. I'm not 100% sure about how to get around this though. Maybe sort the list with class names only, then disable sorting and then insert <all> and <none> in the top of the list. That could work? <all> (or whatever it's resource string value is), will then always be at the top.

Solved with this modification:

---
diff --git a/ide/procedurelist.pas b/ide/procedurelist.pas
index 9e578af..96d8ba9 100644
--- a/ide/procedurelist.pas
+++ b/ide/procedurelist.pas
@@ -431,7 +431,8 @@ begin
       end;
     end;
   finally
-    cbObjects.ItemIndex := 0;   // select <All> as the default
+    { select <All> as the default }
+    cbObjects.ItemIndex := cbObjects.Items.IndexOf(lisPListAll);
     if cbObjects.Text = '' then
       cbObjects.ItemIndex := 1;
   end;

---

Hope someone apply it to svn/git.

Cheers,
Raul Ferriz

__________ Informaci?n de ESET NOD32 Antivirus, versi?n de la base de firmas de virus 4514 (20091016) __________

ESET NOD32 Antivirus ha comprobado este mensaje.

http://www.eset.com

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lazarus.freepascal.org/pipermail/lazarus/attachments/20091016/bd4abe4a/attachment.html>

Reply by email