feat(graphics): Phase 4 - Text Rendering Support#100
Merged
Conversation
Add text rendering capabilities to the graphics stack: - font.rs: Font abstraction wrapping noto-sans-mono-bitmap - FontSize and Weight enums - Font struct with metrics() and glyph() methods - FontMetrics for char_width, char_height, spacing - Glyph wrapper with pixels() iterator for rendering - primitives.rs: Text drawing functions - TextStyle struct with foreground, background, font - blend_colors() for anti-aliased text rendering - draw_char(), draw_text() for character/string rendering - text_width(), text_height(), text_line_height() for measurement - Comprehensive unit tests for text rendering Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add a visual demonstration of the graphics stack that runs during kernel boot in interactive mode: - demo.rs: Draws colorful shapes and text showcasing: - Filled and outlined rectangles in various colors - Circles (filled and outlined, concentric) - Radiating lines in a color gradient - Diagonal line patterns - Text in multiple colors (red, green, blue, yellow) - Highlighted text with background color - Multi-line text rendering - Hooks into kernel_main() after double buffer upgrade - Displays for 3 seconds before clearing and continuing boot - Only runs with --features interactive Also fixes Glyph::pixels() to use rasterized.raster() directly and adds double_buffer_mut() accessor to ShellFrameBuffer. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The stride value from Canvas::stride() is in pixels, not bytes. When calculating row byte offsets, we need to multiply by bytes_per_pixel to get the correct memory location. This fixes the garbled/repeated graphics output where drawing was happening at 1/4 of the correct y-offsets. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
SocketError, SendError, and RecvError can all occur in CI environments where external network access is unreliable. These should SKIP rather than FAIL since they indicate network unavailability, not bugs. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
font.rsmodule wrapping thenoto-sans-mono-bitmapcrate for bitmap font supportprimitives.rswith text drawing functionsChanges
New:
kernel/src/graphics/font.rsFontSizeenum (Size16 currently enabled)Weightenum (Regular currently enabled)Fontstruct withmetrics(),glyph(),replacement_glyph()methodsFontMetricsstruct (char_width, char_height, line_spacing, letter_spacing)Glyphwrapper withpixels()iterator for renderingUpdated:
kernel/src/graphics/primitives.rsTextStylestruct with foreground, background, and font configurationblend_colors()for anti-aliased text renderingdraw_char()- renders single character at positiondraw_text()- renders string with newline handlingtext_width(),text_height(),text_line_height()- text measurementUpdated:
kernel/src/graphics/mod.rsfontmodule typesTest plan
🤖 Generated with Claude Code