This is a list of all comments for UI-DEN-15896-1. Review Summary: No summary ---------------------------------------- File: sources/gui/qml/main.qml Revision Comment by vduong on 24 July 2023, 07:06 https://devapps.diality.us/cru/UI-DEN-15896-1#c18234 Do we need to do this? The previous text binding had the "pairedAddr" in it. Reply by Behrouz NematiPour on 24 July 2023, 12:32 > we wanted to be consistent with the other two for > W: and E: and now to be B: Reply by vduong on 24 July 2023, 12:34 > RESOLVED Revision Comment by msuleiman on 24 July 2023, 11:01 https://devapps.diality.us/cru/UI-DEN-15896-1#c18252 Previous lines are removed Reply by Dara Navaei on 19 October 2023, 10:25 > RESOLVED Revision Comment by vduong on 24 July 2023, 07:14 https://devapps.diality.us/cru/UI-DEN-15896-1#c18236 We should check with Behrouz, but I think we should comment out the entire item instead of just the text property because the Text item still takes up space. For this case, we didn't define a fixed width, but if we did, there will be a gap. Reply by Behrouz NematiPour on 24 July 2023, 12:35 > that is fine with me. > we need to keep this as a debug. Reply by Dara Navaei on 19 October 2023, 10:26 > RESOLVED Revision Comment by msuleiman on 24 July 2023, 11:01 https://devapps.diality.us/cru/UI-DEN-15896-1#c18253 Already checked with Behrous Reply by vduong on 24 July 2023, 12:34 > RESOLVED --- ID: UI-DEN-15896-1 https://devapps.diality.us/cru/UI-DEN-15896-1 Title: UI-DEN-15896_UI MS S102 Statement of Objectives: State: Closed Summary: Author: msuleiman Moderator: msuleiman Reviewers: (5 active, 2 completed*) vduong (*) Behrouz NematiPour (*) Sean Nash jreaume Tiffany Mejia Michael Garthwaite Dara Navaei