ismail-s / ishychat Goto Github PK

Maybe global constants should be set for "CONNECTED", "NOT CONNECTED", "GET NAME", as they are repeated often, both in server.py and in the client's networking.py

The big list of colours in tkinterview.py can be moved into a separate .json file, and read in at runtime.

If someone calls themselves 'black', then their name does not appear as black.

When the chat client window is currently resized, then the entrybox doesn't adjust size as it should.

It can be awkward to install twisted on windows, so maybe it would be better to make the program able to use asyncio as a drop in replacement to twisted.

A send button should be added to the chat client so that if the return key can't be used, then messages can still be sent.

Need to add more tests.

Help string/message in Messages.py needs to be updated to cover the /list and /listusers command.

Application class in tkinterview.py file should have a separate run method to start the application rather than doing this in the constructor method

At the moment, you need to be in the root folder of this project in order to run it ie it can't be run from other places. This is due to absolute file references in the tkinterview and server files.

When using the chat client, the colour of the names of people is often light-coloured such that the name can't be read.
Basically, go through the COLOURS list in TkinterView.py and remove all the ones that are too light.

It would be nice to be able to run the server over asyncio as well as twisted.

The chat client needs to be tested over larger networks than just over a router.

It would be nice to have a way to change names mid conversation in the chat client.