norandomtechie / ece270-simulator

Source code for Web-Based Verilog Simulation Platform in ECE 27000 at Purdue University
25 stars 5 forks source link

Push buttons should describe actions, not states #11

Open kubark42 opened 3 years ago

kubark42 commented 3 years ago

The button block is very good, except for the File vs Workspace button which leads to confusion.

If I look at the below image, is this button telling me I am in File Simulation mode OR that I will go into File Simulation mode if I click it?

Screen Shot 2021-09-13 at 11 25 56 AM

With very few exceptions, buttons should describe an action resulting from the button push. In the case of the above button, a radio button or toggle button would be more appropriate for describing the simulator File/Workspace state.

eduxstad commented 1 year ago

Ideally it should be a dropdown menu.