Skip to content

Conversation

Elektraglide
Copy link
Contributor

Has 4 status LEDs (inside display on 4404) as well as HD activity LED

Comment on lines +32 to +34
<element name="spacer" ref="static_black">
<bounds x="0" y="480" width="640" height="16" />
</element>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You don’t really need this thing as you’ve got explicit bounds for the view, and the LEDs will draw on top of the black background anyway. But if you insist on using it, you’re better off putting it behind the screen rather than on top to ensure it doesn’t slightly crop the bottom of the screen when the floating point view coordinates are quantised to whole pixel values.

Comment on lines +36 to +50
<element name="led_1" ref="led_round">
<bounds x="46" y="484" width="8" height="8" />
</element>
<element name="led_2" ref="led_round">
<bounds x="34" y="484" width="8" height="8" />
</element>
<element name="led_4" ref="led_round">
<bounds x="22" y="484" width="8" height="8" />
</element>
<element name="led_8" ref="led_round">
<bounds x="10" y="484" width="8" height="8" />
</element>
<element name="led_disk" ref="led_rect">
<bounds x="614" y="485" width="16" height="6" />
</element>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it be a good idea to add text labels for the LEDs? As it is, there are three identical-looking round LEDs, the function of which won’t be clear without examining the source. Also, the outputs corresponding to these don’t appear to actually be hooked up?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm guessing these are the LEDs it represents:
image

Full picture at:
http://www.bitsavers.org/pdf/tektronix/44xx/pictures/4404_CPU_T.JPG

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants