Generally, I totally agree that trivial edits are bad for the site. If they're "good" edits then by definition they bring "improvements" but if other edits are left behind, especially if they're meaningful, then it forces someone else to edit, which is more work, another edit history to store all the details for, etc etc. If this is someone else without the privilege and so "suggested" then it's another thing to do in the review queue and a handful of users to review it, the edit ability blocked until it's reviewed, etc.
I am totally behind rejecting tiny edits when there are plenty of other things that should have been done, I really am:
James has approved 248 edit suggestions and rejected 295 edit
suggestions and improved 50 edit suggestions
However, tags are very important, as they specify who will see a question based on user's fave tags and where they are currently perusing. This should be done very quickly as many questions with specific tags on Stack Overflow have a very short time window to get good eyes on it. If it's badly/incorrectly tagged that time window may be missed.
I don't think there's anything wrong with re-tag only if it's worthwhile. In the edits in the case of this question, there are two views to take:
- Maybe there'd have been no harm done taking a bit longer to edit the question as well, as the main tag was "javascript" already on the question so possibly got it's most useful audience anyway
- We don't know who has "angular" and "typescript" as ignored tags, so perhaps this didn't hit that useful audience with those on it
But even just from the importance of tag edits and the fact that the suggested edits by Bharata were legitimate and useful, I think the right thing to have done would have been to "improve".
You also have to consider, "what is a trivial edit"? What is and what isn't is entirely ambiguous, with varying opinions in each case. So sure we should "teach" people why not to do trivial edits and so reject them, but we should also be very careful to not waste people's time, and importantly not fight against the same goal.
Original editor wants to make an improvement. Editor who rejected wants to make an improvement, but rejected original editors edits and re-applied them just because they missed a few things.
Again, I'm against trivial edits and ones which are lazy or more should/could have been done, and this is a very grey area to debate, but we have to be very careful what we choose to reject, and have good reasons with all things considered like the good of the site and its users.
Clearing another thing up:
Looking at Bharata's other edit suggestions, they are not just in it for the rep or badges, and can see they put effort in to many edits in the past.
Here are some examples of suggested edits by Bharata:
The most recent 3
https://stackoverflow.com/review/suggested-edits/20328501
https://stackoverflow.com/review/suggested-edits/20327867
https://stackoverflow.com/review/suggested-edits/20327978
The top 3 from page 4 of their suggested edit lists
https://stackoverflow.com/review/suggested-edits/20282772
https://stackoverflow.com/review/suggested-edits/20282996
https://stackoverflow.com/review/suggested-edits/20283478
Of course no reviewer is going to check a user's previous suggested edits before reviewing their current suggested edit, this is obvious. So Roshana Pitigala could not have known the above info before rejecting, just thought it fair to mention as there's a bit of one-sidedness towards Bharata, as it has been said and agreed that their edit was too trivial.