convict-git / KoraKaagaz

A collaborative whiteboard application as part of CS5617 Software Engineering project at IIT Palakkad
MIT License
13 stars 31 forks source link

Bugs in UI Module #141

Open himanshu2047 opened 3 years ago

himanshu2047 commented 3 years ago

There are some bugs noticed during the server testing -

  1. You are first calling initialise of Processing and Content Module and after that you are initialising your own canvas and everything else, instead you should do the opposite.
  2. There is an error while we are providing the changes to UI, which is NULLPointerException in UI.CanvasController.updateChanges()
  3. You should display the BoardID on the UI somewhere, so that while testing we can know which board they need to connect and generally as well.
  4. Suggestion to make the default color black instead of white as usually the user will start drawing first instead of changing the color and when the color is white he/she will not be able to see anything.

@sai-deekshith16

sai-deekshith16 commented 3 years ago

In the above mentioned bugs all of them are resolved in the last PR except changing the default colour in the colour picker as black. We tried to change it but were not able to.

convict-git commented 3 years ago

In the above mentioned bugs all of them are resolved in the last PR except changing the default colour in the colour picker as black. We tried to change it but were not able to.

Can you open a new issue for it? So I can close this.

sai-deekshith16 commented 3 years ago

In the above mentioned bugs all of them are resolved in the last PR except changing the default colour in the colour picker as black. We tried to change it but were not able to.

Can you open a new issue for it? So I can close this.

I opened a new issue for the colour picker. You can close this.