spatel-code / union

Group project - UnionApp
0 stars 0 forks source link

Minor Bug; multiple instances of the same string. #18

Open gisone-bartolomeo-213704291 opened 4 years ago

gisone-bartolomeo-213704291 commented 4 years ago

Multiple text boxes with the same string can populate the Venn Diagram. Since this is a Venn Diagram only one instance of a string should be used at one time; this is because if a string belongs to multiple groups (i.e. belongs to both left and right circles), then you would need only 'one' instance to be placed in the intersection portion of the diagram. Therefore you would never need more then one instance of the same string.

In this case I populated the graphs with the string 'Test' over and over again (seen in image below). The program should be reworked to stop me from implementing the second 'Test', as a second instance is never needed in a Venn Diagram.

Release 2020-03-09-02-15-38 on Windows 10, Java Version 8.0.2410.7

To replicate, simply add multiple instances of the same string (as I did with the string 'Test').

image

gisone-bartolomeo-213704291 commented 4 years ago

Correction: the build is actually: Release-2020-03-12-23-58-23