search

cjdduarte / MDS_Time_Left

More Decks Stats and Time Left
13 stars 4 forks source link

Delete "Average" and "X minutes more" #25

Closed Okosh50 closed 1 year ago

Okosh50 commented 1 year ago

Hi!

Love what you've done with the add-on, but I wanted to make mine cleaner. Prior to the most recent update, I was able to delete the "Average" cards/min section as well as the "X minutes more" but now I haven't been able to delete the right code as I continually corrupt it. Can you help with this? Thanks!

cjdduarte commented 1 year ago

This is the source code before the modification. Could you change it to the way it's useful?

Link

Send it back to me, and depending on it, I'll adjust the new version to how you want it. Can you modify and send it back to me here?"

Okosh50 commented 1 year ago

Thank you!

I deleted the following lines, which were lines 97-100 on my coding software:

    + "<span style='white-space:nowrap;'>" + _("Average") \
    + ":<br> " + _("%.01f") % (speed) + "&nbsp;" + (_("Cards") + "/" + _("Minutes").replace("s", "")).lower()  \
    + "</span><br><br>" \
    + str(ngettext("%s minute.", "%s minutes.", minutes) % (minutes)).replace(".", "") + "&nbsp;" + _("More").lower() \
Okosh50 commented 1 year ago

Also, I made a couple of cosmetic updates in the newest version of the code in order to: 1) Keep all the stats in 1 line (as opposed to 2 lines on the screen) 2) Make all the stats bolded with equal spacing

These are lines 73-89 for your newest update:

image
cjdduarte commented 1 year ago

I will evaluate here and return

cjdduarte commented 1 year ago

Thank you!

I deleted the following lines, which were lines 97-100 on my coding software:

    + "<span style='white-space:nowrap;'>" + _("Average") \
    + ":<br> " + _("%.01f") % (speed) + "&nbsp;" + (_("Cards") + "/" + _("Minutes").replace("s", "")).lower()  \
    + "</span><br><br>" \
    + str(ngettext("%s minute.", "%s minutes.", minutes) % (minutes)).replace(".", "") + "&nbsp;" + _("More").lower() \

I will evaluate if I leave this part optional also in a future version. Thanks.