-
Notifications
You must be signed in to change notification settings - Fork 825
feat: added remaining Wave Generator functionality #2852
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Reviewer's GuideReorganized the Wave Generator screen layout, integrated Play functionality to launch Oscilloscope/Logic Analyzer with device-connection checks, and extended localization with two new string keys. Sequence diagram for Play button launching Oscilloscope/Logic Analyzer with connection checksequenceDiagram
actor User
participant WaveGeneratorScreen
participant ScienceLab
participant Navigator
participant ScaffoldMessenger
User->>WaveGeneratorScreen: Tap Play button
WaveGeneratorScreen->>User: Show menu (Oscilloscope/Logic Analyzer)
User->>WaveGeneratorScreen: Select instrument
WaveGeneratorScreen->>ScienceLab: isConnected()
alt Device connected
WaveGeneratorScreen->>Navigator: Navigate to instrument screen
else Device not connected
WaveGeneratorScreen->>ScaffoldMessenger: Show not connected message
end
Class diagram for updated WaveGeneratorScreen and localization keysclassDiagram
class WaveGeneratorScreen {
+String sineWaveCircuit
+String squareWaveCircuit
+String oscilloscopeIcon
+String logicAnalyzerIcon
}
class AppLocalizations {
+String sharingMessage
+String delete
}
class AppLocalizationsEn {
+String sharingMessage
+String delete
}
AppLocalizations <|-- AppLocalizationsEn
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
Build successful. APKs to test: https://github.com/fossasia/pslab-app/actions/runs/17072875229/artifacts/3799657402 |
6950e90 to
8c9dd3a
Compare
Does the following additions:
Screenshots / Recordings
untitled.mp4
untitled1.mp4
Summary by Sourcery
Enhance the Wave Generator screen with functional Play controls allowing users to open the oscilloscope or logic analyzer when the hardware is connected, update layout structure for consistent UI, and extend localization support with new strings.
New Features:
Enhancements:
Chores: