-
Notifications
You must be signed in to change notification settings - Fork 0
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
fixing some bugs on datepicker that were failing the test #151
Conversation
Reviewer's Guide by SourceryThis PR fixes a few bugs related to the datepicker component, primarily focusing on issues encountered during testing. The changes include updates to the tests themselves to skip specific scenarios on mobile devices, adjustments to the calendar rendering logic to correctly manage navigation button visibility, and minor updates to the demo files. Sequence diagram for updated calendar rendering processsequenceDiagram
participant C as AuroCalendar
participant U as UtilCalRender
participant UC as UtilCal
participant D as Dropdown
C->>U: setFirstRenderableMonthDate()
C->>UC: assessNavigationButtonVisibility()
C->>D: Find closest dropdown element
Note over C,D: Check for dropdown or dropdownbib
Class diagram showing calendar component structureclassDiagram
class AuroCalendar {
+boolean visible
+render()
}
class CalendarUtilities {
+assessNavigationButtonVisibility(elem)
}
class RangeDatepicker
AuroCalendar --|> RangeDatepicker
AuroCalendar --> CalendarUtilities
note for CalendarUtilities "assessNavigationButtonVisibility is now public"
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 @sun-mota - I've reviewed your changes - here's some feedback:
Overall Comments:
- Please provide explanation for why the mobile-related tests are being skipped rather than fixed. If there are legitimate reasons, document them in the test file.
- The PR title and description should be more specific about what bugs were fixed and what changes were made. This helps with future maintenance and debugging.
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 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.
🎉 This PR is included in version 1.6.0-beta.9 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Alaska Airlines Pull Request
Related to #85
Before Submitting this pull request:
Development
sectionnote: all pull requests require at least one linked ticket
Ready For Review
, all ticket's linked underDevelopment
must have their status changed toReady For Review
as wellBy submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license and I have performed a self-review of my own update.
Summary by Sourcery
Tests: