From 427ac1368ff78c0eb76bab67ccd2f1f28ef873f7 Mon Sep 17 00:00:00 2001
From: John Bintz <hello@theindustriousrabbit.com>
Date: Tue, 4 Oct 2022 08:08:15 -0400
Subject: [PATCH] this is close enough

---
 main   | Bin 8204 -> 8976 bytes
 main.c | 193 ++++++++++++++++++++++++++++++++++++++++++++++++++-------
 2 files changed, 170 insertions(+), 23 deletions(-)

diff --git a/main b/main
index b1040bdb147b05e23b707fc0ab2bd7c871b655c5..a11839ace799be79e8b033a8fc5e89643cce2d3f 100755
GIT binary patch
delta 4654
zcma)A4Qx}_6+Z91o5XdU#7*7Eq$as@-OP3$wu>T~qO5%}Y4Kd+1RAk0p<-vj4xJ%x
zNKhJO_>m%pCR7C;tx$y~R8>$-7Da#%L##_HG^Gels6r^JP=hKaewO~ILsQBi*>|6v
zG?XyS(RuIQbMHC#+;hKk&VAXVjl4Wk9WL8NBzcLbgh;tbRJJQ}efzX0;-O#u04}Km
z#8#rE9P1_C1V&RruK#59*vwsDD{J$`mD77Wyg3#-mRCBu+PSK9l<JtJ`!waMyNM){
z;zwteNA)zxRp4Ej9c8hLkZ_Z23wg+~*@>|WzGh`?UtYP|JgJ=HJK{(0c@lC!^R+7D
zpiAIh<arh=N-!}V&a*_8+<RwgDo>_Lvt0?ITfX1Y0k+=oO1y%#U)=GUE^9DWQ$~%o
z(o@n|V^C_zu|$wO8uybY?Nl?$Nv^r!16X><r?K^J(HiC%%(6r>$}!$e9$z;x)*e4o
z>51woTl`EF%Tj{zATdvdwLML~jB=61gtyk^mGPpDR$;Tb4;IKM<6LDOX~wm9ZX|xD
zVp>ctPOOUNm2+vE*f_*52$4V(&b*qoxiiWYF%u>QqS$Vl3xnM%Tb~xZ!^N>7H_VM6
zo!*tG5*)~mrY(>?E9CG#iLp*z7n{g!kM|pA<!YSSMtZCwa-B+>CuyIWVI!D&8jku-
zoEGf?_b|A-!Tk!{+u&|P|G?fcsFK0C22NM_2-m>X-L056-1;psj7(u98a@&}k|dk%
z*V<b7DsD+PMh3_mEKgFo3NPmzZLR5gx7Iu<y^y6|6KGJ}5~Z|otqtyX!08TV&q}e0
z1jYM#<!b#H0I-ql9l~t5(~+)qYXuKXzaS+1-BiKL(-yIeZD&U-{4kZa7EFYJL+D?F
zPhoPru#<at!TK(36)e*O$I-t%zdyV9$XVdJt6)Io_#f|*2S(I4e@m*^zfsGcs@oTg
zP)#tSOpzztL$z6ZP$T~05Z$0E;Y}mKoHA=P2HE10yTa+lK~`HOHKu_RDF)f<gaxfN
z#s&7=6TcRH$VyZj%}9+YE1t^@XUQwicZ!j$RMag~ZAg_GBctIXK?*kM3ROfhQU)yx
z`QdTdZ25w!<TVbK1X>t{f@Ck%$o8T}b}ZLhTu50^(a-{lR6{R?BK8wS!$n0U3o6>P
zfTEC4Br}U~tkh}ZvD_KC9$Ly_D*(=q>nW#Ol~j+)yeD_0Y#vn#)vGLYYB=47{e(bj
zkg(f*kqm20QpGLc&k;T%dgf}6@{DrVol$Ox6ME--y};;Y%(3}0l9zEjp&8E*3&nC1
zF`K~Hx+2DGZwD&`qMjY1N5~~w;SY(6_s@n%Dc8t@h;tONw^ZFAWKD=T3J|fy0S|YQ
z?j{frlWq;B2r~|3XWaD8c_2Ls&ZIEzo%7k5_|e>STau=47pLjQXRtLNn`kkCZz{GG
z6hHcJVVH6QqLT%MB5Z^*u}er@nP-f%DD!FLI$tVq^sCVpis+Hi=QZGN4Ls#cQrUWh
zQqJk7uXpUzJ<>mi_wTqqd`bU$_)?y^d|{~pX_kY$ppfZ0kLfzqbX~*q0I9Sb#{aFO
zHGza6pk}(?3vbav?l<SCb90!Olbe4&=7iAP1d+(#rXSA_i+uyy7n_K&c>iA~MN+U{
z^<|hx!g(^_0?!616h_P>>E0pC{TS3acStP-zgY8yI98So<p>eERyY(cZ18Km90f&6
zkn{mUys_xox9c^8q=jnmtRxqnR^lSm!yl1bL^ML6-b?@?aQ$WiP=i)La)2arJ2t@e
zeY5o_e`UqDv@~1p5wQF!C$Q_;P_H4XTnJAq!qfGD=&6N7|1ZgN|4TAx<+)Yv9%?E8
zY8IiM4Hy8|y{drG={C;>zDoF_08hS|)@I4>rOIBCJ_$(Cms4?GR*Z>H(fdZsDEjU+
z3T_l-;0u&p$2vW2V>+%LcT?j-QsNTN<O)#=R~#1T&Wd}(p~P7(%o6TL;Fvfs^h}(a
zrikM@ou$}BX7o$s<hc_rMvLX|>slk%rDgsea&0HR#9W%_TCZ(n2ddm3dP8_u!;~D+
zXnL7x6Jg+Ng6MMnW^!dI^bFObpjlpp&ESqArvHTZX#i`_b|8d~V$KqB0_`?oC=AMa
zwB}pn4z`g4{&R3U<SSm)Q$)jNv@|6KAUz0fw%G4B#vb80$qR)w1$916Qqi-`^+MaF
z=*qMwNq5d9YKvmkv-#qFzrh=NN9&v~cmeHY9|G-m;SI!B)7GB;Ex-iX>re!~<QABo
z8{h(q7tIu@Nx15y*93=%N}okq{fL~sa<G|PXmPHs1YC{JL?)X#93YVwMeQwQ%dBGN
zNfbZvM&7(HR|?~BRCZzBY_=`KMMNsiMe~v;?VnW!Xf2tA){n5_8OZ}%2e9yY)MkCY
zn$odgMUrN4c~GtukK$04;K`8iCdiwlAGRYaG!`<$o&E48D#y%#nKN^9_LS-<L?|sH
zJ<6)MN3e(F$NhP!fy=n+R))1Cm7K!WChS(3$0OZD9<7&ezV+_#&9}f&3!HxpUmm`k
zXASS_z7_1{ES=IjOJTqZiDg{LK@C;dp(ZPA<w|c8+Fj=JHD9x&7CCFs4kc;o6n3M<
zCde#Y^J_#2@;$l3N05>(!4HMonK)M5&X`Ty*^p7E%*6At;0hF>oaKu6ot~O4JhSIh
z(pxAxMW>#}5sA4X4A0LiH}(S1N?JtFl`Tus?OI%yt$1G)xs<Mmqpb%{#1X(aM^Rq`
zYUGWmo-?N&Hm4pB4b99Tow{HvV)vy<$t<790h4f-LsSI5>HhQsygXCDiz>l8U8r|W
zA|dI!W>b=%c**GR(_VALEq0B&{ak|bzoeNLRjar?{^u~N**K=}N}m>mGkqKpdo7U{
zicv~#6a&vpKS}!6_ht&ZDuV7rtUNBYDD4SnRApT$7X5w73Uk}}>i9^2I`U|t7ad7m
zU4+WXx_DWjF5Vx1V_j=AKD1<4V0YTi-$cu4d|6R#lN-RW6d1=VOC?!eZCLCvnY9~h
z?e1!&8y`T__q)Agmxi}_S4=9|m6OWO6K~rslApCPO}ZV>;&-M;h&Vo0f|R6htisCp
z8-PeP(xM$zz1tXlEM;NciT42XZX-}tjqe-OKSz=z4)@z}Zdqe()dx#icd8Vf1ipR1
z@<muCNtVxJ6WxU4T?>#vK3yHnvuxVFMhB*{hWdC!Rt}ul_W1iHRV<ORs;>Yw=K_>)
z;Cttw;a}vES)UPEv_@tL<KUu973(XU*S&F|bpSME)YSNwv7xkM?zZ2#cMOXUfQAJA
zf)y+7kROZB7u6xU(-;(@J0B3$A-eOSqProw$M`~s?s-6TH$?Y5RCENQ`;FyKLUjKF
zq9YL9|4`8b5ItzT_oP3^4n81y0HOy?QA?2Ge@3Esy$$C`_$iZkNTNklTY*O@2>r)u
zKpv1xWS;`h0kRgx2)xQ|Xyd1R54wfMWP49GIr#If#tPRja?PDR{e8S`)8Jr|w{P0K
zv6sk;zwc<N&@Hz>r&w$Dhko%VW|*`3gtkDcXG@<svI{pS`U&qnvd@Z<G=HsT<1eo2
z=T6NHnhn@Y^yz)PqbBbHc%RSlj~mI_I^*fZTXV(z+_#JHn>^any@iK2ZRs1NzV3~i
zwsiGx?d#pt)4#Ph+1a_NFYue5%|x9WGnqi+)@=i{fEK=lKU@-?!N=m_2t7kC{9nO!
z1GF2A4+|DPp`k5!V|aV<?#6rfiSfl&1h><;R$E_Pbsi_Oer5&&41eaY>Z-x({I4qr
zWd%i9juZ|-*6>E%kQ+e_AQc3yG7etmAQ#994K@%CPSHW_XreC(=S_nII0OJLgARlC
RfwIQ2x*D0;jSF?!e*tmNIK=<}

delta 3845
zcmZ`+e{2)i9sk}v+qlLg#&aVVLO8CT7rU;VB9w+HqZc<So-vL~5epM)ZA5+9b_Fj$
zm7)xr6j6loN8q4Btm;@5CDCLt1gOe7#N4T>(uhe>6;;uM8iZH^UJ4P47-f*`^Ufy9
zfI9l#z4v|J&-Z<Qz3;vByw<JsRD~^fh-emMCSo^<EO#Q4`)5575B>ama4FD1ru7&x
zYA2}?jCD!T`}ylba}RtiY`_<1)}yVpS=M_tFAnV3ECs{?OAb;wpTg?oMiNMfpPBn<
zG^dj21TQeZ+B*V-O!l4RA=f)@tX2D(nEP0s)iq5qmvkV0=AkDk1{Lg63-W*~NqN>=
zMws||IL`(%Bp;nyR%uFGGCPt)Mc;4eB)g#33Q`5jj~sY2Cn_*jVGezZ@VsEx_X*8e
zHW(t0BKgUqx&s<BNs3%c1WQjt@Pf=oU6KYVYeZutwUNiyMkbbze`X(z=F;}~XO$N+
zlr%{pGI=yM@MH36Ok%w}T3hq1wrrz~+idt@3TmuY3b3;(_KIhFpy&)2E(<!NdFE2>
z+;PCGIV2c`c%5pOH5TBKFewPBK~>@g%K>&l<-ES~S|{WN<7Z|MC7qlDtV6YeXXiM)
zAv?QWS{WP9?vL-)FN#&j+1pfcDKbfxrYTB5=?#bu!v6@*7IlNW1Kg|NUIRA<?z@;z
zA02`!5u7#P><ITu3b<o(3*@?5Zj%Ws39LlJ{o(!;+3NgCqD9&$S=EL}2i1m_fOiEl
zSuN3`dSwOM+{>W2N`A~XM@cmyw04AFL(pBUEhEInlN8@8Rlc4(ivobzR&F+Y)YKYT
zDI?c$gujg{q$R4=*k(;@3uF?itz;q$?8JNxVhWSvB~Knc04t)}ILnB@Im}}V=Z`k;
zxrn;nQ8Hl3;g27X2S#|CZc~8!uh*EKUzrF+Xjw>O3G#$@QBB4XQb>BGlWvfcM^jIl
z+#>7I_nB6hH}jzDADL>LbVo&<2o_+gl2(lD`eoD0&${>=Sb=Jy8r@0T@SM@z<_&AB
z`ASA8>lUe|E<kr8g>Zj}LX9~_6%kGFVq_&hA}$&uUnoG=^pDKJX4YC4B8NpG`zs1L
zuw9_M5qqkl`ll$G&~p|j+V+^D#<C*wQx(1T6h%d@NMuR<ti^5AqirZMS+`n(ttjx0
zbGfuz4$y=rz;YK3r0t#n6}<s=_|q;m0e_01i4eit7*1(VitMLQf39#ppBcL?MoDaz
zHP*m0bYj@cYxEn)*nJvZV;y)-upFL6b~u(DkJ)*RZ7tWBZC$8>SJX?L^bA$=QFwqP
z{$ABdY>7f1DAvlgw>t0^#~LN#DwW7HvnUTaMGulF_C{F&lAAG|nUm?nFpf|ZoJMZk
ziQ&v#{7iN>k)qj|ve=mZ6>QDN#+!}8Hx%2;iu(-18USr9DTK0Su8eyL)RhIsxXYOK
z$fUHI*U>Ku%@oN+3TqYA-6ljTlOnTS3b1TOrtc0M%Xx%<cO5@4*)^K`u4^>U_V~hd
z3vIRolmi_$Lf2`8?w}F6?zOx~XJGu7tu0A31YT-J_-kQQEBEm6@EbA{^D^7GOm<;@
z41wajK#~hAT*qd&K*wU^F;jf+-==s|u=bz_chWUH57Gpl-Bb+Y%%teyX~<pzZJ$q4
zIhNLKTU#QNLcM<%Wvlm-^dh;o^5%EB-QIIRZEvr`v&xji!7r%w{4Eex`;wS@-=XRZ
z)xAJoG&}1XIn@sCpRmahygy0NoVH)U=p<xbBsU_#o9Q&tlkq4`MUq1aa!I>DjyH1Y
z%-T*9bl=r4E|$Y|4aj9BoKK+%%<S{C;(j5zUiF}^&L(S&$lpy?Jt`V#@8e@#vX{=&
zqB#1LBQnYAfBz4}v~l;Yy*m+Aa;(I`-zHgM(ul-Q#RxwYeg&;_OQ;jKfS1S}{xcjD
zd6&V+6Iv{7)W140y#6-f^EB7zeJe=VO7o)OmX1h<@%bvq^LIEq@c$<S%r)f*oX!yZ
zDstaI*HZM~uaT%brDQ$00pGe|h?F763sl)o!sh`&vXoES-=&S*XqYe@8`lco!o7v3
z*77DA2wDQmWK$&Cj1RyM6!!5v-I7GyQ7Xd<&cJaT(^GPQb$W!GH~?Pp=KCLZ-FzRM
zgC)+ty1wc9CeOM*%=unrA7|*(+;$7E!DPL}poJ<N&=ThkNvypwl_hf+FETg!nrOmc
zk+}}zVv6oxhtE4><79N4g@@aGns<$GKfVsf;AiQx%9F!CtH#F`M<L<iDZ9kEh$?Rc
z<=UFP_o!iP&1Fj`-<&9Qi*o3P0pP;CPYjp(#G{61_C_WbRi)_eCfsmrDf*8o!wa|o
zgpp;QBADeWYgz+?8cLFP|7l2mC@a#hXbaL-i?r!WD}R)|KZt|iMBllaj!TPjhaM%-
zPm;Otfm&;<$lM9sv9^@$8blrP`+|*s#2>iPRK_5ENFC(;OFf5}TatMM@d9+)IJRsN
z_PQI+P;9*IYJ6|}xAE`k{2wYe2+d+|(j5)3i)mBzgS1W7#9rz3cu$a8^B7?k?MYvK
z4n2%rlokb7a_H8UCR}+1EqGXUNbh0PrU#c+*@c5(SWAraiKP}n6#l5UI%6Ws>wQkS
zO5}G@+Vi`kL!({q*1kF=7S>ORqvyYMS_MBFFe$=JP>J84?IGg##t@}w`j@ROd_yF?
z3O&ey23yb_uC$dElBc8qCi`7gPBw(Qtt&->Btr%s59xib-qoy-wqP=dYbaR02>T*M
z*013{A#GcbcHq^jXr2wJj!ijK$&lXpY(x}qod5FKPs~m>n6?ErpbowWQqt8lB~I!4
zmRF1HvVL;;CXo&5Q_D3cyIe}y!!e|L1T?Duy4qjQMpf7R#cTUfzWE4flvh-!;??`)
z$L0&69DYR~=FlroK*s<zK3Y$OUIp~J?q31u^(UZL0lof%yWRkFN<YG(Q%^u|06O)9
z&}l$t^x_JCmd!i?od$HqfLcQo{|k}5F@Zc1eryU4fscrmU&6xz!n|S_GzvNg8UW7)
zvX$0IBwFId7(a&p$5;`A!scC>)V`nW*7cg7dG_#eFuiN%9z&>^tVECTTJ`#sp_fbR
zh7f;1G5#0hkNKZw@Qz@PIxPP(dTP~j{jV!`E8E}Jv|z*TcRP%g3XN#lJ)-Bv@IDD@
zfK#m{kaHSi&g%e;f!vrkoF9{HI2LXC^;O;~$39q)pPK`~gg@iw=+PzFpB1lS=`tb%
z4Z?8{j1js)hd~EH_y`HmEWjZl1A-+&8#F8iA)Mj}s25{|l?ot!IsX*ILkoSh1C#&-
YLHHSA#lN>?4pju+BSx%EACZ;+0cQVoeE<Le

diff --git a/main.c b/main.c
index 901fa91..3013afc 100644
--- a/main.c
+++ b/main.c
@@ -63,9 +63,24 @@ struct Screen *screen;
 struct Window *window;
 struct Gadget *windowGadgets;
 
-//struct NewWindow aboutWindowLayout = { };
-struct Window *aboutWindow;
-struct Gadget *aboutWindowGadgets;
+#define ABOUT_WINDOW_WIDTH (240)
+#define ABOUT_WINDOW_HEIGHT (50)
+
+struct NewWindow aboutWindowLayout = {
+  40, 40,
+  ABOUT_WINDOW_WIDTH, ABOUT_WINDOW_HEIGHT,
+  0, 1,
+  IDCMP_REFRESHWINDOW | IDCMP_CLOSEWINDOW | BUTTONIDCMP,
+  WFLG_DRAGBAR | WFLG_DEPTHGADGET | WFLG_CLOSEGADGET | WFLG_ACTIVATE,
+  NULL,
+  NULL,
+  "About Pizza Timer",
+  NULL,
+  NULL,
+  ABOUT_WINDOW_WIDTH, ABOUT_WINDOW_HEIGHT,
+  ABOUT_WINDOW_WIDTH, ABOUT_WINDOW_HEIGHT,
+  WBENCHSCREEN
+};
 
 // Fonts!
 struct TextAttr Topaz80 = { "topaz.font", 8, 0, 0 };
@@ -88,6 +103,11 @@ struct Gadget *timerGadget, *hourSlider, *minuteSlider, *secondSlider;
 
 // gadtools menus
 // https://wiki.amigaos.net/wiki/GadTools_Menus
+
+// Why the special struct, constants and struct instantiation?
+// Because this way, when we get a menu message from Intuition,
+// we can look up the menu item selected by ID, rather than
+// looking it up by position in the menu tree structure.
 struct MenuData {
   int id;
 };
@@ -118,6 +138,8 @@ struct timeval currentSystemTime;
 // get that bell
 #define BELL_FILENAME "bell.8svx"
 
+APTR bellSound = NULL;
+
 // our business logic
 // for how long should I cook this pizza?
 unsigned int uiHours = 0;
@@ -229,10 +251,8 @@ struct Gadget *buildUI(void) {
   currentGadget = CreateContext(&glist);
 
   ng.ng_LeftEdge = WINDOW_CHROME_WIDTH;
-  // TODO: constantize these
   ng.ng_TopEdge = 11;
   ng.ng_Height = 12;
-
   ng.ng_Width = 0;
   ng.ng_GadgetText = NULL;
   ng.ng_TextAttr = &Topaz80;
@@ -258,6 +278,7 @@ struct Gadget *buildUI(void) {
   ng.ng_TextAttr = &Topaz80;
   ng.ng_Height = 12;
 
+  // start/stop button
   ng.ng_Width = (WINDOW_WIDTH - WINDOW_CHROME_WIDTH * 2) / 2;
   ng.ng_TopEdge += 18;
   if (timerIsRunning) {
@@ -276,6 +297,7 @@ struct Gadget *buildUI(void) {
       TAG_END
       );
 
+  // reset button
   ng.ng_LeftEdge += (WINDOW_WIDTH - WINDOW_CHROME_WIDTH * 2) / 2;
   ng.ng_GadgetText = "_Reset";
   ng.ng_GadgetID = RESET_BUTTON_ID;
@@ -289,6 +311,7 @@ struct Gadget *buildUI(void) {
       TAG_END
       );
 
+  // hours slider
   ng.ng_LeftEdge = 85;
   ng.ng_Width = (WINDOW_WIDTH - WINDOW_CHROME_WIDTH * 2) - 85 + 4;
   ng.ng_TopEdge += 12;
@@ -314,6 +337,7 @@ struct Gadget *buildUI(void) {
       TAG_END
       );
 
+  // minutes slider
   ng.ng_TopEdge += 12;
   ng.ng_GadgetText = "Mins:   ";
   ng.ng_GadgetID = MINUTES_SLIDER_ID;
@@ -331,6 +355,7 @@ struct Gadget *buildUI(void) {
       TAG_END
       );
 
+  // seconds slider
   ng.ng_TopEdge += 12;
   ng.ng_GadgetText = "Secs:   ";
   ng.ng_GadgetID = SECONDS_SLIDER_ID;
@@ -351,6 +376,9 @@ struct Gadget *buildUI(void) {
   return glist;
 }
 
+/**
+ * Set the text of the timer and redraw the necessary gadgets.
+ */
 void setTimerText(void) {
   // Only change the timer widget text if the time is different
   // from the last render. This prevents unnecessary renders and potential
@@ -422,8 +450,6 @@ void clearUI(void) {
 void handleToggleTimer(void) {
   timerIsRunning = !timerIsRunning;
 
-  // TODO: don't reset the timer when it's stopped/started
-
   if (timerIsRunning) {
     // http://amigadev.elowar.com/read/ADCD_2.1/Includes_and_Autodocs_2._guide/node04FA.html
     GetSysTime(&currentSystemTime);
@@ -441,9 +467,12 @@ void handleToggleTimer(void) {
 
     timerStarted = TRUE;
 
-    // start the async timer
+    // start the timer
     startTimer();
   } else {
+    // put the sliders back.
+    // it causes a lot of UI flashing to update these as the timer runs,
+    // so only do it once we stop the timer.
     GT_SetGadgetAttrs(
       hourSlider,
       window,
@@ -487,6 +516,93 @@ void handleResetTimer(void) {
   clearUI();
   renderUI();
 }
+//
+// To keep this simple, the about window will block
+int openAboutWindow(void) {
+  struct Window *aboutWindow;
+  struct NewGadget ng;
+  struct Gadget *currentGadget;
+  struct Gadget *glist;
+  struct IntuiMessage *iMessage;
+
+  ULONG windowSignal;
+  BOOL closeAbout = FALSE;
+  aboutWindow = OpenWindow(&aboutWindowLayout);
+
+  if (!aboutWindow) {
+    return 1;
+  }
+
+  currentGadget = CreateContext(&glist);
+
+  ng.ng_LeftEdge = WINDOW_CHROME_WIDTH + 4;
+  ng.ng_TopEdge = 15;
+  ng.ng_Height = 10;
+  ng.ng_Width = 0;
+  ng.ng_GadgetText = NULL;
+  ng.ng_TextAttr = &Topaz80;
+  ng.ng_VisualInfo = visualInfo;
+
+  ng.ng_GadgetID = NULL;
+  ng.ng_Flags = PLACETEXT_IN;
+
+  currentGadget = CreateGadget(
+    TEXT_KIND,
+    currentGadget,
+    &ng,
+    GTTX_Text, "Topaz's Pizza Timer",
+    GTTX_CopyText, TRUE,
+    TAG_DONE
+  );
+
+  ng.ng_TopEdge += 10;
+  currentGadget = CreateGadget(
+    TEXT_KIND,
+    currentGadget,
+    &ng,
+    GTTX_Text, "By John Bintz",
+    GTTX_CopyText, TRUE,
+    TAG_DONE
+  );
+
+  ng.ng_TopEdge += 10;
+  currentGadget = CreateGadget(
+    TEXT_KIND,
+    currentGadget,
+    &ng,
+    GTTX_Text, "theindustriousrabbit.com",
+    GTTX_CopyText, TRUE,
+    TAG_DONE
+  );
+
+  AddGList(aboutWindow, glist, -1, -1, NULL);
+  RefreshGList(glist, aboutWindow, NULL, -1);
+  GT_RefreshWindow(aboutWindow, NULL);
+
+  windowSignal = 1L << aboutWindow->UserPort->mp_SigBit;
+
+  while (!closeAbout) {
+    Wait(windowSignal);
+
+    while (!closeAbout && (iMessage = GT_GetIMsg(aboutWindow->UserPort))) {
+      switch (iMessage->Class) {
+        case IDCMP_CLOSEWINDOW:
+          closeAbout = TRUE;
+          break;
+        case IDCMP_REFRESHWINDOW:
+          GT_BeginRefresh(window);
+          GT_EndRefresh(window, TRUE);
+          break;
+      }
+    }
+  }
+
+  RemoveGList(aboutWindow, glist, -1);
+  FreeGadgets(glist);
+  CloseWindow(aboutWindow);
+
+  return 0;
+}
 
 /**
  * Process a single Intuition message.
@@ -514,8 +630,8 @@ void handleIntuitionMessage(struct IntuiMessage *iMessage) {
 
           break;
       }
-
       break;
+
     // We picked a menu item.
     case IDCMP_MENUPICK:
       // https://en.wikibooks.org/wiki/Aros/Developer/Docs/Libraries/GadTools
@@ -526,11 +642,20 @@ void handleIntuitionMessage(struct IntuiMessage *iMessage) {
       switch (menuData->id) {
         // I originally tried to use MENU_QUIT.id here
         // https://stackoverflow.com/questions/14069737/switch-case-error-case-label-does-not-reduce-to-an-integer-constant
+        // This is why there's a bunch more overhead for building
+        // the menu userdata.
         case MENU_QUIT_ID:
           terminated = TRUE;
           break;
+        case MENU_ABOUT_ID:
+          // yes, this blocks, deal with it
+          // it also returns non-zero if something goes wrong.
+          // in that case, kill the program
+          if (openAboutWindow()) terminated = TRUE;
+          break;
       }
       break;
+
     // We've moved the mouse. In slider gadget talk, we've changed
     // the value of the slider.
     case IDCMP_MOUSEMOVE:
@@ -554,8 +679,8 @@ void handleIntuitionMessage(struct IntuiMessage *iMessage) {
       if (rerenderTimer) setTimerText();
       break;
 
-      // bye bye
     case IDCMP_CLOSEWINDOW:
+      // bye bye
       terminated = TRUE;
       break;
 
@@ -568,12 +693,15 @@ void handleIntuitionMessage(struct IntuiMessage *iMessage) {
   }
 }
 
-void endTimer(void) {
-  // TODO: play an included IFF 8SVX sound
-  APTR bellSound = NULL;
+
+/**
+ * Start playing a bell sound, and start a timer to tear down
+ * the bell data once it's done.
+ */
+void startBellSound(void) {
   struct dtTrigger myTrigger;
   struct MsgPort *TimerPort;
-  ULONG soundPlayResult, timerSignal;
+  ULONG soundPlayResult;
 
   if (bellSound = NewDTObject(BELL_FILENAME, DTA_GroupID, GID_SOUND, TAG_END)) {
     myTrigger.MethodID = DTM_TRIGGER;
@@ -592,7 +720,30 @@ void endTimer(void) {
     TimerIO->tr_time.tv_micro = 0;
     SendIO((struct IORequest *)TimerIO);
   }
+}
 
+/**
+ * Tear down the bell once the timer runs out.
+ */
+void waitForBellToFinish(void) {
+  struct MsgPort *TimerPort;
+  ULONG timerSignal;
+
+  if (bellSound) {
+    TimerPort = TimerIO->tr_node.io_Message.mn_ReplyPort;
+
+    timerSignal = 1L << TimerPort->mp_SigBit;
+    Wait(timerSignal);
+
+    DisposeDTObject(bellSound);
+  }
+}
+
+/**
+ * Handle the timer running out of time.
+ */
+void endTimer(void) {
+  startBellSound();
   DisplayBeep(screen);
 
   uiHours = uiMinutes = uiSeconds = 0;
@@ -604,12 +755,7 @@ void endTimer(void) {
   clearUI();
   renderUI();
 
-  if (bellSound) {
-    timerSignal = 1L << TimerPort->mp_SigBit;
-    Wait(timerSignal);
-
-    DisposeDTObject(bellSound);
-  }
+  waitForBellToFinish();
 }
 
 void handleTimerMessage(void) {
@@ -648,7 +794,6 @@ void clearMenu(void) {
 }
 
 int main() {
-  // TODO: Menu bar with about menu and change the title to match the time
   struct IntuiMessage *iMessage;
   struct MsgPort *TimerPort;
   ULONG windowSignal, timerSignal, foundSignals;
@@ -671,14 +816,16 @@ int main() {
     return 1;
   }
 
+  // http://amigadev.elowar.com/read/ADCD_2.1/Includes_and_Autodocs_3._guide/node03F8.html
+  SetWindowTitles(window, (UBYTE *) ~0, "Topaz's Pizza Timer - theindustriousrabbit.com");
+  buildMenu();
+
   // these create the bit mask for Wait() to listen to events on
   windowSignal = 1L << window->UserPort->mp_SigBit;
   timerSignal = 1L << TimerPort->mp_SigBit;
 
   renderUI();
 
-  buildMenu();
-
   while (!terminated) {
     // http://amigadev.elowar.com/read/ADCD_2.1/Libraries_Manual_guide/node02EB.html
     // http://www.amigadev.elowar.com/read/ADCD_2.1/Includes_and_Autodocs_2._guide/node038A.html