TanukiSharp / MHArmory

Armor Set Search application for Monster Hunter: World
MIT License
26 stars 8 forks source link

[feature] Trim trailing whitespace loadout name #20

Closed lyuha closed 5 years ago

lyuha commented 5 years ago

How about trim trailing whitespace at loadout name? I thinks that don't need the whitespace.

TanukiSharp commented 5 years ago

Usually in names I prefer to apply as less processing as possible. If people want to use leading and trailing white spaces, why not ? If they don't want, they just don't input them.

Do you have a particular case that causes problems ?

lyuha commented 5 years ago

I don't have any problem.

Usually, don't need trailing whitespace. People distinguish loadouts by names. It implies trailing space doesn't make a difference and change hard to identify them. Is there any difference between hammer and hammer?

Otherwise, leading spaces can help indentation.

TanukiSharp commented 5 years ago

The question is more "why would you add unnecessary white spaces to your loadout names if you don't want to ?" The loadout names are something personal, it's not data you grab from remote source, it is yours, written by you, for you.

lyuha commented 5 years ago

Ok, I got it. The loadout name is only personal space.

Thanks for the reply.