t_backoff wrote:if we are going to ajax, it needs to be throughout the entire core, not bits and parts. For example, why are we using trigger_error in some parts, but ajax in others?
Choose one, the other, or none at all.
...because sometimes you need a trigger_error() and other places make more sense to display a modal box or a flash message or something. We aren't trying to turn this into ajaxBB. We are just trying to implement AJAX technology into places in which it makes sense.
It is better to get the basic AJAX framework merged and then work on features bit by bit, testing and reviewing each one separately, than to try and fit everything into one huge, unreadable PR that will inevitably be merged without proper review.
Think of it like making a new software. If you try to implement everything at once, you get very sketchy, disorganized and orphaned (i.e. unused but retained because you forgot to remove it) code. If you focus your efforts on one feature at a time and give each one the attention it deserves, you get a much cleaner and more maintainable code base.