Page 1 of 3 1 2 3 LastLast
Results 1 to 15 of 31

Thread: Reacties op NLDCalendar

  1. #1
    Misschien is een screenshot leuk?

  2. #2
    die kun je vinden in de hoofdthread
    Last edited by eagle00789; 20-Jul-04 at 00:30.
    Check out ackspace.nl for a dutch hackerspace

  3. #3
    Hey,

    Leuk componentje Eagle.

    Ik heb nog een idee voor een optie, het zou handig zijn als je kunt invullen waar de kleuren voor staan.
    En die zouden dan bijv. kunnen verschijnen als tooltip.

    Graag je reactie.

    Greetz: Delphi Man

    [Edit]
    Je reactie word je 200ste post
    [Edit]

  4. #4
    persoonlijk gezien had ik dit idee ook al. maar ik ben nog bezig om te kijken hoe ik dit het beste kan doen. ik weet namelijk nog niet hoe ik zo'n tooltip kan laten verschijnen. dat ben ik nog aan het uitzoeken

    [edit]mijn 200ste post[/edit]
    Check out ackspace.nl for a dutch hackerspace

  5. #5
    Originally posted by eagle00789
    ik weet namelijk nog niet hoe ik zo'n tooltip kan laten verschijnen. dat ben ik nog aan het uitzoeken
    Laat dit een hulp zijn
    Attached Files Attached Files
    DeX 3 Delphi := The ease of VB with the power of C; Zoekt en gij zult vinden

  6. #6
    Thx dit is wat ik nodig had. ik ga dit meteen ff inbouwen in het component zodat je bij het toevoegen van een kleur ook een hint kunt opgeven
    Check out ackspace.nl for a dutch hackerspace

  7. #7
    5th member of nldelphi
    Join Date
    Mar 2001
    Location
    Alkmaar
    Posts
    2,127
    eagle,

    Het ziet er leuk / grappig uit, toch echter even een paar 'opmerkingen' hierover.

    Code:
    if AYear <> Year  then
      raise EDateRangeError.Create
           ('The supplied date is not covered by this calendar');
    
    if AMonth <> Month  then
      raise EDateRangeError.Create
           ('The supplied date is not covered by this calendar');
    zou je mischien kunnen vervangen voor het onderstaande, scheelt immers weer
    een paar regels en het wordt overzichtelijker.

    Code:
     if (AYear <> Year) or (AMonth <> Month) then
       raise EDateRangeError.Create
         ('The supplied date is not covered by this calendar');

    Verder viel mij op dat het grid pas gekleurd wordt indien je weer met je muis op
    het grid klikt nadat je de color gezet hebt.
    Dit is feitelijk gezien niet hellemaal netjes, als ik zeg dat er een kleurtje gezet moet
    worden dan verwacht ik ook dat dit gebeurt en niet pas nadat ik op het grid klik.

    Code:
    if TheText = dayText then
          begin
            Found := True;
            ColorGrid[Col, Row] := Color;
                   {write the color of this cell to the color grid array}
            UpdateCalendar; (* Update de kleurtjes voor richard on the fly. *)
          end;

    Verder wil ik je alleen even er op attenderen, je gebruikt in je check de Year en
    Month property van het Calender object, initieel als de calender gecreërd wordt
    worden deze property's gelijk gezet met de datum van vandaag, dit staat 'haaks' op
    het feit dat je designtime deze property's kan instellen waarbij je verwacht dat
    de calender runtime deze datum overneemt. Niets is hellaas minder waar.
    Om de datum correct te zetten dien je de CalenderDate property te zetten, op dat
    moment worden jouw property's wel correct gezet.

    My 2 Cents.

    Richard
    RLD

  8. #8
    Beste Richard, jou tips zijn verwerkt, behalve het feit dat jij zegt:
    Originally posted by Richard
    Verder viel mij op dat het grid pas gekleurd wordt indien je weer met je muis op
    het grid klikt nadat je de color gezet hebt.
    Dit is feitelijk gezien niet hellemaal netjes, als ik zeg dat er een kleurtje gezet moet
    worden dan verwacht ik ook dat dit gebeurt en niet pas nadat ik op het grid klik.
    dit is niet waar. hij word toch direct gekleurd.

    Verder zeg jij nog:
    Originally posted by Richard
    Verder wil ik je alleen even er op attenderen, je gebruikt in je check de Year en
    Month property van het Calender object, initieel als de calender gecreërd wordt
    worden deze property's gelijk gezet met de datum van vandaag, dit staat 'haaks' op
    het feit dat je designtime deze property's kan instellen waarbij je verwacht dat
    de calender runtime deze datum overneemt. Niets is hellaas minder waar.
    Om de datum correct te zetten dien je de CalenderDate property te zetten, op dat
    moment worden jouw property's wel correct gezet.
    dit snap ik helaas niet. zou je dit nog iets beter kunnen uitleggen
    Check out ackspace.nl for a dutch hackerspace

  9. #9
    De hint zit in deze versie nog niet erbij

    Geupdate zodat 1 van de problemen die richard heeft ontdekt is opgelost (het direct kleuren van een cel)

    Alle code van het origineel zit nu in deze calender

    Ook de Helpfile (gegenereerd door BDoc) is geupdate

    De source kun je plukken vanaf: hier
    Last edited by eagle00789; 05-Jan-04 at 11:17.
    Check out ackspace.nl for a dutch hackerspace

  10. #10
    5th member of nldelphi
    Join Date
    Mar 2001
    Location
    Alkmaar
    Posts
    2,127
    Originally posted by eagle00789
    dit is niet waar. hij word toch direct gekleurd.
    Dit snap ik helaas niet. zou je dit nog iets beter kunnen uitleggen
    Hellaas voor je maar het is toch waar... althans in mijn voorbeeld op mijn
    machine wel.

    Enfin voor de duidelijkheid een sample bijgesloten... kijk ook even in de NLDCalender
    unit welke ook in de directory zit.

    Richard

    PS. bijgesloten een RAR bestand met een ZIP extensie, even renamen indien nodig
    naar RAR en dan even uitpakken.
    Attached Files Attached Files
    RLD

  11. #11
    ik zie nu idd beide problemen. probleem 1 is al door jou opgelost. maar ik moet kijken hoe ik probleem 2 kan oplossen.
    Last edited by eagle00789; 01-Jan-04 at 17:22.
    Check out ackspace.nl for a dutch hackerspace

  12. #12
    Ik heb er nog geen oplossing voor, maar jou 2de probleem ligt zelfs al in de originele calender.

    Ook heb ik de source geupdate, zodat mijn component geen derive meer is van TCalendar, maar alle code van het origineel zit nu in die van mij.
    Check out ackspace.nl for a dutch hackerspace

  13. #13
    5th member of nldelphi
    Join Date
    Mar 2001
    Location
    Alkmaar
    Posts
    2,127
    Originally posted by eagle00789
    maar alle code van het origineel zit nu in die van mij.
    En de reden daarvan is ?

    Je bedoelt eerder: "ik heb het orgineel gekopieerd en er een tweetal methods aan
    toegevoegd".

    Ik snap echter niet waarom je alle code wil 'overhevelen' van het orgineel naar die
    van jouw, want het origneel maakt ook gebruik van anchestor classes dus compleet
    onafhangelijk zijn van andere source code is er toch niet bij.
    Dus bij mijn inziens... is het 'overkill'.

    En inderdaad, het laatste punt deed zich ook al voor in het orgineel.

    Richard
    RLD

  14. #14
    Ik heb dat gedaan omdat dan de uiteindelijke overhead dezelfde grote heeft als het origineel. ook staat het wat netter vind ik persoonlijk
    Check out ackspace.nl for a dutch hackerspace

  15. #15
    5th member of nldelphi
    Join Date
    Mar 2001
    Location
    Alkmaar
    Posts
    2,127
    Welke overhead ? of eerder.. wat bedoel je met overhead in de zin van dit component ?

    Richard
    RLD

Page 1 of 3 1 2 3 LastLast

Thread Information

Users Browsing this Thread

There are currently 1 users browsing this thread. (0 members and 1 guests)

Similar Threads

  1. NLDCalendar Versiehistorie
    By eagle00789 in forum NLDCalendar
    Replies: 3
    Last Post: 15-Apr-08, 11:01
  2. Replies: 16
    Last Post: 14-Aug-06, 23:29
  3. Replies: 16
    Last Post: 18-Sep-05, 12:23
  4. Reacties op artikel 'Rave artikel 1: De basis'
    By Marcel in forum De website
    Replies: 14
    Last Post: 26-Jul-05, 10:05
  5. Meer idee?½n gevraagd voor NLDCalendar
    By eagle00789 in forum NLDCalendar
    Replies: 2
    Last Post: 23-Jul-04, 17:09

Bookmarks

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •