-
Notifications
You must be signed in to change notification settings - Fork 2
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
Calendar picker. #10 #26
Conversation
Manually merged
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.
I also have a silly question, but isn't the grid layout a possibility here?
location.go
Outdated
} | ||
|
||
func calendar(l *location) { | ||
l.calendar = container.New(NewCalendarLayout(32), calendarObjects(l)...) |
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.
Would it be possible to move all the function and logic related to the Calendar to its own file? I would also expect a l.calendar := newCalendar().
location.go
Outdated
b := container.New(layout.NewBorderLayout(nil, nil, l.monthPrevious, l.monthNext), | ||
l.monthPrevious, l.monthNext, container.NewCenter(l.monthLabel)) | ||
|
||
l.dateContainer = container.NewVBox(b, l.calendar) |
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.
Maybe also an newCalendarPopup() with showAtPosition function.
location.go
Outdated
monthNext *widget.Button | ||
|
||
dateContainer *fyne.Container | ||
calendar *fyne.Container |
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.
Not sure that you actually need to store this here.
I initially tried to use |
This will matter when the text changes
Deviations from design:
Small inset of date button due to button padding
Colours
Calendar popup position is the same regardless of scroll position or button clicked, to be changed?
No dropdown arrow icon next to date button