Tim Abbott

@mahimg if you're up for it, it'd be awesome if you worked on the restructuring discussed above. I think this improvement is pretty important and would love to get it...

@alexisopsasnick can you clean up the commit history a bit? Check out our [GitHub guide](https://zulip.readthedocs.io/en/latest/git-guide.html) and [commit guidelines](https://zulip.readthedocs.io/en/latest/contributing/version-control.html) for more details.

I merged the first commit, after some small changes to whitespace.

I merged the navbar fix. @vaidap I'm not sure the `x`s should be hidden for ARIA; won't that make it really hard to use those exit buttons?

For the uploaded images, I think the right answer might be to have `alt=""`. The title of the image is already included in the message in the form of the...

I looked at reviewing this, but ended up spending the last hour rewriting it because of several issues: * The way the live-update was done involved a lot of calls...

Thanks for fixing the test failures; merged. Thanks @jai2201!

I like `links.py` as well.

I suggested the name `arguments.py`, because the only reason we need globals in `bugdown` is because `python-markdown` doesn't support passing arguments (and thus data) into the functions for formatting directly....

I kinda think `add_a` and `add_embed` belong in `links.py`, not a new `elements.py`, since I don't see a clear boundary between those. Finally, I'm concerned about whether `db_data` and `current_message`...