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.

Sign up for GitHub

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

Jump to bottom

add autowrapping to Yast2::Popup class #1122

Merged
merged 10 commits into from
Feb 11, 2021
Merged

add autowrapping to Yast2::Popup class #1122

merged 10 commits into from
Feb 11, 2021

Conversation

jreidinger
Copy link
Member

how example looks like:

autowrap5

@coveralls
Copy link

coveralls commented Dec 8, 2020 •
edited
Loading

Coverage Status

Coverage decreased (-0.001%) to 40.487% when pulling 159c6c1 on popup2_autowrap into bddcabc on master.

joseivanlopez
joseivanlopez reviewed Dec 11, 2020
View reviewed changes
package/yast2.changes Outdated Show resolved Hide resolved
library/general/src/lib/yast2/popup.rb Outdated Show resolved Hide resolved
joseivanlopez
joseivanlopez approved these changes Feb 11, 2021
View reviewed changes
Copy link
Contributor

@joseivanlopez joseivanlopez left a comment

Choose a reason for hiding this comment

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

LGTM

@jreidinger jreidinger merged commit 33a082e into master Feb 11, 2021
@jreidinger jreidinger deleted the popup2_autowrap branch February 11, 2021 10:23
@yast-bot
Copy link
Contributor

✔️ Public Jenkins job #282 successfully finished
✔️ Created OBS submit request #871161

@yast-bot
Copy link
Contributor

✔️ Internal Jenkins job #144 successfully finished
✔️ Created IBS submit request #235782

This was referenced Feb 12, 2021
Merged
Merged
Merged
Merged
This was referenced Feb 15, 2021
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
This was referenced Feb 17, 2021
Merged
Merged
@ancorgs ancorgs mentioned this pull request Feb 18, 2021
Merged
This was referenced Feb 18, 2021
Merged
Merged
This was referenced Feb 18, 2021
Merged
Merged
@jreidinger jreidinger mentioned this pull request Feb 18, 2021
Merged
@ancorgs ancorgs mentioned this pull request Feb 19, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joseivanlopez joseivanlopez joseivanlopez approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jreidinger @coveralls @yast-bot @joseivanlopez