#1077 Change title of tempreatureView #1094

Merged

a24julot merged 1 commit from #1077-Change-title-of-temperatureView- into team_1_week_7

2026-05-22 10:10:35 +00:00

b24jonel commented

2026-05-21 13:57:46 +00:00

Collaborator

Moved the sensor ID from the title of the view to above the temperature inside the view.

Skärmbild 2026-05-21 155404.png

18 KiB

b24jonel added the

needs review

label

2026-05-21 13:57:46 +00:00

b24jonel added 1 commit

2026-05-21 13:57:47 +00:00

Moved sensor id 6749d8d04a

Sensor id now appears above the temprature

a24julot

2026-05-22 08:15:20 +00:00

added the
frontEnd
label
requested review from f21marfo

f21marfo approved these changes

2026-05-22 09:13:19 +00:00

f21marfo left a comment

Collaborator

Review on #1094

Whats done

Tested on Windows 11 25H2 using Edge

App starts
- Tested adding a Temperature View card and tested it with light mode, dark mode and different themes. Appears consistent with screenshot.

Inspected following files

TemperatureView.js
- Indentation/code styling
  - Minor changes, fundamental indentations and styling unchanged.
TemperatureView.module.css
- Indentation/code styling
  - Checks out.
- CSS standard
  - Checks out.
- Naming convention
  - Checks out.
- Comments
  - Checks out.

Conclusion

Everything looks good and seems to work, I say this is ready for merge!

# Review on #1094 ## Whats done ### Tested on Windows 11 25H2 using Edge + App starts + Tested adding a Temperature View card and tested it with light mode, dark mode and different themes. Appears consistent with screenshot. ### Inspected following files + TemperatureView.js + Indentation/code styling + Minor changes, fundamental indentations and styling unchanged. + TemperatureView.module.css + Indentation/code styling + Checks out. + CSS standard + Checks out. + Naming convention + Checks out. + Comments + Checks out. ## Conclusion + Everything looks good and seems to work, I say this is ready for merge!