Thanks for the feedback Aaron! :) I've disabled it. I think the annoying part really is the status (e.g. Request, Closed etc) in the tag, and I am wondering if a tag with just line numbers like "(N Loc)" would be better. But I'm not really sure about the trade-off here.
- Eric On Fri, Sep 30, 2016 at 3:05 PM Aaron Ballman <aa...@aaronballman.com> wrote: > On Fri, Sep 30, 2016 at 2:04 AM, Eric Liu via cfe-commits > <cfe-commits@lists.llvm.org> wrote: > > I've switched the default email format to be plain text only now. This > > option should be per-user configurable, but somehow it is not shown in > the > > "Settings"; I'll try if I can make the option personalized. > > Thank you for working on this upgrade! > > Is there a way that we can remove the [Request, X lines] tag from the > email headers on phab patches? While it's neat to understand how large > a patch is, you can get a rough feel for that information by looking > at the patch contents. Unfortunately, the extra text means we lose > information from the subject line when viewing emails from a smaller > screen (like a mobile device), so I'm not certain that the utility > warrants the cost (at least, to me). > > ~Aaron >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
