search

dyne / tomb

the Crypto Undertaker
https://dyne.org/software/tomb
GNU General Public License v3.0
1.32k stars 151 forks source link

Modest extras/gtomb update #478

Closed nerun closed 1 year ago

nerun commented 1 year ago
  1. replaced soft with hard tabs to maintain code standard;
  2. inserted a "wait" statement in the dig, forge, lock and open functions;
  3. spelling standardization (canceled to cancelled);
  4. simplified the call to MONMORT icon inside zenity functions;
  5. zenity warning function changes;
  6. fixed missing dots.

About Update "1"

My previous PR #465 (merged) used soft tabs, because my editor uses soft tabs, but gtomb itself uses hard tabs. Now uniform.

About Update "2"

Some messages like Success: Tour tomb has been dug in ... was appearing BEFORE the tomb was actually dug. With a wait statement the message appears AFTER the tomb has been created. Similar situations occurred in forge, lock and open functions.

About Update "3"

Most of text is in english British, Canadian, Australian (cancelled) but a few times "cancelled" appears spelled as "canceled" (USA english).

About Update "4"

Transformed --window-icon="$MONMORT" into a variable Icon="--window-icon="$MONMORT"". Zenity functions now just call the variable: $Icon. Text/code economy and easier to update.

About Update "5"

Just a simplification. All warning messages but one, have title "Warning", that one had title "gtomb". I have moved title to inside the function.

About Update "6"

Self explanatory.

nerun commented 1 year ago

Fixed PR #477

jaromil commented 1 year ago

thanks again! well needed update