Inconsistent email header option naming

ABTech / tracker

Carnegie Mellon Activities Board Technical Committee Tracker

abtech.org

22 stars 29 forks source link

Inconsistent email header option naming #567

Open tomas-goncalves opened 1 month ago

tomas-goncalves commented 1 month ago

Currently when you load a page with an email there is an option to "View All Headers". Clicking this changes the text to "Hide Extra Headers". Clicking the toggle again changes the text to "View Hidden Headers". The name should probably be the same all the times (maybe extra headers as it is independent of the current rendering state). I can PR a fix later today if "Extra Headers" sounds ok.

NoRePercussions commented 1 month ago

Sounds good. You could consider just "Show Headers" and "Hide Headers" as well; it is not perfectly accurate but it is descriptive. I'm not picky about what term you settle on.