-
Notifications
You must be signed in to change notification settings - Fork 3
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
Ahmed/chore add quill UI #6
Ahmed/chore add quill UI #6
Conversation
@sourcery-ai review |
Reviewer's Guide by SourceryThis pull request implements a new trading UI using Quill UI components. It replaces the old DerivTrading component with a new implementation that includes a trading panel for setting trade parameters and a chart for displaying market data. The update also includes UI changes to the overall layout and styling. Class diagram for the trading components structureclassDiagram
class ChartProps {
+string symbol
+boolean chartStatus
+boolean showChart
+onChartStatusChange(status: boolean)
+onSymbolChange(symbol: string)
}
class ChartSettings {
+boolean assetInformation
+boolean countdown
+boolean isHighestLowestMarkerEnabled
+string language
+string position
+string theme
}
class TradingPanelProps {
+number duration
+number stake
+boolean allowEquals
+DurationTabValue selectedDurationTab
+StakeTabValue selectedStakeTab
+onDurationChange(value: string)
+onStakeChange(value: string)
+onAllowEqualsChange(e: Event)
+onDurationTabChange(tab: DurationTabValue)
+onStakeTabChange(tab: StakeTabValue)
}
class TabProps~T~ {
+Array~TabItem~ tabs
+T activeTab
+onChange(value: T)
}
TradingPanelProps ..> TabProps
Chart ..> ChartSettings
Chart ..> ChartProps
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.
Hey @ahmed-deriv - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟡 General issues: 1 issue found
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Summary by Sourcery
New Features: