Skip to content
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

Date and time widgets #891

Merged
merged 3 commits into from Jan 18, 2019
Merged

Date and time widgets #891

merged 3 commits into from Jan 18, 2019

Conversation

teclator
Copy link
Contributor

@teclator teclator commented Jan 17, 2019

This PR adds CWM widgets for Date and Time fields.

Note: It seems that both are now supported in ncurses and this check is not needed probably since

Example screenshot (Y2STYLE=installation.qss)

Ncurses example screenshot

dateandtimefieldsncurses

@coveralls
Copy link

coveralls commented Jan 17, 2019

Coverage Status

Coverage increased (+0.01%) to 27.927% when pulling e5972e4 on date_and_time_widgets into 8503626 on master.

@@ -0,0 +1,69 @@
# Simple example to demonstrate object API for CWM
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like copy and paste

Copy link
Contributor Author

@teclator teclator Jan 17, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was based on the popup example so, basically copied & modified example, yes 😄

https://github.com/yast/yast-yast2/blob/master/library/cwm/examples/popup.rb#L1

@teclator teclator merged commit b6638bd into master Jan 18, 2019
@teclator teclator deleted the date_and_time_widgets branch January 18, 2019 08:50
@yast-bot
Copy link
Contributor

✔️ Internal Jenkins job #82 successfully finished
✔️ Created IBS submit request #181965

@yast-bot
Copy link
Contributor

✔️ Public Jenkins job #76 successfully finished
✔️ Created OBS submit request #667692

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants