Merge ~jibel/ubiquity:advanced_features_dialog into ubiquity:master
- Git
- lp:~jibel/ubiquity
- advanced_features_dialog
- Merge into master
Proposed by
Jean-Baptiste Lallement
| Status: | Merged |
|---|---|
| Merged at revision: | 5a2667c9678e03f9a154cee3306b5204adb2a8d6 |
| Proposed branch: | ~jibel/ubiquity:advanced_features_dialog |
| Merge into: | ubiquity:master |
| Diff against target: |
561 lines (+308/-121) 6 files modified
debian/changelog (+2/-0) debian/ubiquity.templates (+32/-0) gui/gtk/stepPartAsk.ui (+211/-106) tests/run-pep8 (+1/-0) tests/test_gtkui.py (+3/-0) ubiquity/plugins/ubi-partman.py (+59/-15) |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Didier Roche-Tolomelli (community) | Approve | ||
| Steve Langasek | Needs Information | ||
|
Review via email:
|
|||
Commit message
New design for advanced features
A new dialog has been added to the guided partitioning page for advanced features. LVM, encrypted LVM and ZFS have been moved to this new dialog.
Specification:
https:/
Description of the change
To post a comment you must log in.
Revision history for this message
| Steve Langasek (vorlon) : | # |
review:
Needs Information
Revision history for this message
| Jean-Baptiste Lallement (jibel) : | # |
Revision history for this message
| Didier Roche-Tolomelli (didrocks) wrote : | # |
Revision history for this message
| Jean-Baptiste Lallement (jibel) wrote : | # |
@didrocks thanks for the review.
I'll have a look at the next/back behaviour separately.
For the margins, they are the same than on the advanced partitioning dialog boxes but different than the partman confirmation dialog boxes for instance. I'll leave it like this and align all the margins in a separate MP.
for reference https:/
Revision history for this message
| Didier Roche-Tolomelli (didrocks) wrote : | # |
with those additional info + a chat on IRC, I'm fine with the testing and it seems to align with ubiquity code style and mpt design.
+1 on my side. Just think about the followup commit to refresh templates. Thanks!
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
| 1 | diff --git a/debian/changelog b/debian/changelog | |||
| 2 | index 5d1a3dd..8a152ab 100644 | |||
| 3 | --- a/debian/changelog | |||
| 4 | +++ b/debian/changelog | |||
| 5 | @@ -1,6 +1,8 @@ | |||
| 6 | 1 | ubiquity (20.04.1) UNRELEASED; urgency=medium | 1 | ubiquity (20.04.1) UNRELEASED; urgency=medium |
| 7 | 2 | 2 | ||
| 8 | 3 | * script/zsys-setup: fix tag name from org.zsys to com.ubuntu.zsys. | 3 | * script/zsys-setup: fix tag name from org.zsys to com.ubuntu.zsys. |
| 9 | 4 | * Added an advanced features dialog to the partitioning page. LVM, LVM with | ||
| 10 | 5 | encryption and experimental ZFS support have been moved to this dialog. | ||
| 11 | 4 | 6 | ||
| 12 | 5 | -- Jean-Baptiste Lallement <jean-baptiste.lallement@ubuntu.com> Mon, 28 Oct 2019 09:07:57 +0100 | 7 | -- Jean-Baptiste Lallement <jean-baptiste.lallement@ubuntu.com> Mon, 28 Oct 2019 09:07:57 +0100 |
| 13 | 6 | 8 | ||
| 14 | diff --git a/debian/ubiquity.templates b/debian/ubiquity.templates | |||
| 15 | index 57c5ca3..f56ac69 100644 | |||
| 16 | --- a/debian/ubiquity.templates | |||
| 17 | +++ b/debian/ubiquity.templates | |||
| 18 | @@ -1449,6 +1449,38 @@ Default: false | |||
| 19 | 1449 | Description: for internal use; can be preseeded | 1449 | Description: for internal use; can be preseeded |
| 20 | 1450 | This will set up ZFS and configure it for zsys. | 1450 | This will set up ZFS and configure it for zsys. |
| 21 | 1451 | 1451 | ||
| 22 | 1452 | Template: ubiquity/text/advanced_features_dialog | ||
| 23 | 1453 | Type: text | ||
| 24 | 1454 | _Description: Advanced Features | ||
| 25 | 1455 | |||
| 26 | 1456 | Template: ubiquity/text/advanced_features_radio_none | ||
| 27 | 1457 | Type: text | ||
| 28 | 1458 | _Description: None | ||
| 29 | 1459 | |||
| 30 | 1460 | Template: ubiquity/text/advanced_features_desc | ||
| 31 | 1461 | Type: text | ||
| 32 | 1462 | _Description: None selected | ||
| 33 | 1463 | |||
| 34 | 1464 | Template: ubiquity/text/advanced_features_selected | ||
| 35 | 1465 | Type: text | ||
| 36 | 1466 | _Description: Advanced features... | ||
| 37 | 1467 | |||
| 38 | 1468 | Template: ubiquity/text/advanced_features_none_selected | ||
| 39 | 1469 | Type: text | ||
| 40 | 1470 | _Description: None selected | ||
| 41 | 1471 | |||
| 42 | 1472 | Template: ubiquity/text/advanced_features_lvm_selected | ||
| 43 | 1473 | Type: text | ||
| 44 | 1474 | _Description: LVM selected | ||
| 45 | 1475 | |||
| 46 | 1476 | Template: ubiquity/text/advanced_features_crypto_selected | ||
| 47 | 1477 | Type: text | ||
| 48 | 1478 | _Description: LVM and encryption selected | ||
| 49 | 1479 | |||
| 50 | 1480 | Template: ubiquity/text/advanced_features_zfs_selected | ||
| 51 | 1481 | Type: text | ||
| 52 | 1482 | _Description: ZFS selected | ||
| 53 | 1483 | |||
| 54 | 1452 | Template: ubiquity/text/verified_crypto_label | 1484 | Template: ubiquity/text/verified_crypto_label |
| 55 | 1453 | Type: text | 1485 | Type: text |
| 56 | 1454 | _Description: | 1486 | _Description: |
| 57 | diff --git a/gui/gtk/stepPartAsk.ui b/gui/gtk/stepPartAsk.ui | |||
| 58 | index bb89fc6..f734558 100644 | |||
| 59 | --- a/gui/gtk/stepPartAsk.ui | |||
| 60 | +++ b/gui/gtk/stepPartAsk.ui | |||
| 61 | @@ -1,6 +1,169 @@ | |||
| 62 | 1 | <?xml version="1.0" encoding="UTF-8"?> | 1 | <?xml version="1.0" encoding="UTF-8"?> |
| 63 | 2 | <interface> | 2 | <interface> |
| 64 | 3 | <!-- interface-requires gtk+ 3.0 --> | 3 | <!-- interface-requires gtk+ 3.0 --> |
| 65 | 4 | <object class="GtkDialog" id="advanced_features_dialog"> | ||
| 66 | 5 | <property name="can_focus">False</property> | ||
| 67 | 6 | <property name="title" translatable="yes">Advanced Features</property> | ||
| 68 | 7 | <property name="resizable">False</property> | ||
| 69 | 8 | <property name="modal">True</property> | ||
| 70 | 9 | <property name="window_position">center-on-parent</property> | ||
| 71 | 10 | <property name="type_hint">dialog</property> | ||
| 72 | 11 | <child internal-child="vbox"> | ||
| 73 | 12 | <object class="GtkBox" id="advanced_features_box1"> | ||
| 74 | 13 | <property name="can_focus">False</property> | ||
| 75 | 14 | <property name="orientation">vertical</property> | ||
| 76 | 15 | <property name="spacing">24</property> | ||
| 77 | 16 | <child internal-child="action_area"> | ||
| 78 | 17 | <object class="GtkButtonBox" id="advanced_features_buttonbox"> | ||
| 79 | 18 | <property name="can_focus">False</property> | ||
| 80 | 19 | <property name="layout_style">end</property> | ||
| 81 | 20 | <child> | ||
| 82 | 21 | <object class="GtkButton" id="advanced_features_cancelbutton"> | ||
| 83 | 22 | <property name="label">_Cancel</property> | ||
| 84 | 23 | <property name="use_action_appearance">False</property> | ||
| 85 | 24 | <property name="visible">True</property> | ||
| 86 | 25 | <property name="can_focus">True</property> | ||
| 87 | 26 | <property name="can_default">True</property> | ||
| 88 | 27 | <property name="receives_default">True</property> | ||
| 89 | 28 | <property name="use_underline">True</property> | ||
| 90 | 29 | </object> | ||
| 91 | 30 | <packing> | ||
| 92 | 31 | <property name="expand">False</property> | ||
| 93 | 32 | <property name="fill">False</property> | ||
| 94 | 33 | <property name="position">0</property> | ||
| 95 | 34 | </packing> | ||
| 96 | 35 | </child> | ||
| 97 | 36 | <child> | ||
| 98 | 37 | <object class="GtkButton" id="advanced_features_okbutton"> | ||
| 99 | 38 | <property name="label">_OK</property> | ||
| 100 | 39 | <property name="use_action_appearance">False</property> | ||
| 101 | 40 | <property name="visible">True</property> | ||
| 102 | 41 | <property name="can_focus">True</property> | ||
| 103 | 42 | <property name="can_default">True</property> | ||
| 104 | 43 | <property name="has_default">True</property> | ||
| 105 | 44 | <property name="receives_default">True</property> | ||
| 106 | 45 | <property name="use_underline">True</property> | ||
| 107 | 46 | </object> | ||
| 108 | 47 | <packing> | ||
| 109 | 48 | <property name="expand">False</property> | ||
| 110 | 49 | <property name="fill">False</property> | ||
| 111 | 50 | <property name="position">1</property> | ||
| 112 | 51 | </packing> | ||
| 113 | 52 | </child> | ||
| 114 | 53 | </object> | ||
| 115 | 54 | <packing> | ||
| 116 | 55 | <property name="expand">False</property> | ||
| 117 | 56 | <property name="fill">False</property> | ||
| 118 | 57 | <property name="position">0</property> | ||
| 119 | 58 | </packing> | ||
| 120 | 59 | </child> | ||
| 121 | 60 | <child> | ||
| 122 | 61 | <object class="GtkBox" id="advanced_features_box2"> | ||
| 123 | 62 | <property name="visible">True</property> | ||
| 124 | 63 | <property name="can_focus">False</property> | ||
| 125 | 64 | <property name="orientation">vertical</property> | ||
| 126 | 65 | <child> | ||
| 127 | 66 | <object class="GtkRadioButton" id="advanced_features_radio_none"> | ||
| 128 | 67 | <property name="label" translatable="yes">None</property> | ||
| 129 | 68 | <property name="visible">True</property> | ||
| 130 | 69 | <property name="can_focus">True</property> | ||
| 131 | 70 | <property name="receives_default">True</property> | ||
| 132 | 71 | <property name="active">True</property> | ||
| 133 | 72 | <property name="draw_indicator">True</property> | ||
| 134 | 73 | <property name="margin_top">9</property> | ||
| 135 | 74 | <signal name="clicked" handler="advanced_features_option_changed" swapped="no"/> | ||
| 136 | 75 | </object> | ||
| 137 | 76 | <packing> | ||
| 138 | 77 | <property name="expand">False</property> | ||
| 139 | 78 | <property name="fill">True</property> | ||
| 140 | 79 | <property name="position">0</property> | ||
| 141 | 80 | </packing> | ||
| 142 | 81 | </child> | ||
| 143 | 82 | <child> | ||
| 144 | 83 | <object class="GtkRadioButton" id="use_lvm"> | ||
| 145 | 84 | <property name="label" translatable="yes">Set up the system as an LVM volume group</property> | ||
| 146 | 85 | <property name="visible">True</property> | ||
| 147 | 86 | <property name="can_focus">True</property> | ||
| 148 | 87 | <property name="receives_default">False</property> | ||
| 149 | 88 | <property name="active">True</property> | ||
| 150 | 89 | <property name="draw_indicator">True</property> | ||
| 151 | 90 | <property name="margin_top">9</property> | ||
| 152 | 91 | <property name="group">advanced_features_radio_none</property> | ||
| 153 | 92 | <signal name="clicked" handler="advanced_features_option_changed" swapped="no"/> | ||
| 154 | 93 | </object> | ||
| 155 | 94 | <packing> | ||
| 156 | 95 | <property name="expand">False</property> | ||
| 157 | 96 | <property name="fill">True</property> | ||
| 158 | 97 | <property name="position">1</property> | ||
| 159 | 98 | </packing> | ||
| 160 | 99 | </child> | ||
| 161 | 100 | <child> | ||
| 162 | 101 | <object class="GtkCheckButton" id="use_crypto"> | ||
| 163 | 102 | <property name="label" translatable="yes">Encrypt the volume group for security</property> | ||
| 164 | 103 | <property name="visible">True</property> | ||
| 165 | 104 | <property name="sensitive">False</property> | ||
| 166 | 105 | <property name="can_focus">True</property> | ||
| 167 | 106 | <property name="receives_default">False</property> | ||
| 168 | 107 | <property name="draw_indicator">True</property> | ||
| 169 | 108 | <property name="margin_left">24</property> | ||
| 170 | 109 | </object> | ||
| 171 | 110 | <packing> | ||
| 172 | 111 | <property name="expand">False</property> | ||
| 173 | 112 | <property name="fill">True</property> | ||
| 174 | 113 | <property name="position">2</property> | ||
| 175 | 114 | </packing> | ||
| 176 | 115 | </child> | ||
| 177 | 116 | <child> | ||
| 178 | 117 | <object class="GtkLabel" id="use_crypto_desc"> | ||
| 179 | 118 | <property name="visible">True</property> | ||
| 180 | 119 | <property name="sensitive">False</property> | ||
| 181 | 120 | <property name="can_focus">False</property> | ||
| 182 | 121 | <property name="margin_left">48</property> | ||
| 183 | 122 | <property name="xalign">0</property> | ||
| 184 | 123 | <property name="yalign">0</property> | ||
| 185 | 124 | <property name="use_markup">True</property> | ||
| 186 | 125 | <property name="wrap">True</property> | ||
| 187 | 126 | <property name="label" translatable="yes"><span size="small">You'll choose a security key in the next step</span></property> | ||
| 188 | 127 | </object> | ||
| 189 | 128 | <packing> | ||
| 190 | 129 | <property name="expand">False</property> | ||
| 191 | 130 | <property name="fill">True</property> | ||
| 192 | 131 | <property name="position">3</property> | ||
| 193 | 132 | </packing> | ||
| 194 | 133 | </child> | ||
| 195 | 134 | <child> | ||
| 196 | 135 | <object class="GtkRadioButton" id="use_zfs"> | ||
| 197 | 136 | <property name="label" translatable="yes">Use ZFS (experimental)</property> | ||
| 198 | 137 | <property name="visible">True</property> | ||
| 199 | 138 | <property name="can_focus">True</property> | ||
| 200 | 139 | <property name="receives_default">False</property> | ||
| 201 | 140 | <property name="active">True</property> | ||
| 202 | 141 | <property name="draw_indicator">True</property> | ||
| 203 | 142 | <property name="margin_top">9</property> | ||
| 204 | 143 | <property name="group">advanced_features_radio_none</property> | ||
| 205 | 144 | <signal name="clicked" handler="advanced_features_option_changed" swapped="no"/> | ||
| 206 | 145 | </object> | ||
| 207 | 146 | <packing> | ||
| 208 | 147 | <property name="expand">False</property> | ||
| 209 | 148 | <property name="fill">True</property> | ||
| 210 | 149 | <property name="position">4</property> | ||
| 211 | 150 | </packing> | ||
| 212 | 151 | </child> | ||
| 213 | 152 | </object> | ||
| 214 | 153 | <packing> | ||
| 215 | 154 | <property name="expand">False</property> | ||
| 216 | 155 | <property name="fill">True</property> | ||
| 217 | 156 | <property name="position">1</property> | ||
| 218 | 157 | </packing> | ||
| 219 | 158 | </child> | ||
| 220 | 159 | </object> | ||
| 221 | 160 | </child> | ||
| 222 | 161 | <action-widgets> | ||
| 223 | 162 | <action-widget response="-6">advanced_features_cancelbutton</action-widget> | ||
| 224 | 163 | <action-widget response="-5">advanced_features_okbutton</action-widget> | ||
| 225 | 164 | </action-widgets> | ||
| 226 | 165 | </object> | ||
| 227 | 166 | |||
| 228 | 4 | <object class="GtkAlignment" id="stepPartAsk"> | 167 | <object class="GtkAlignment" id="stepPartAsk"> |
| 229 | 5 | <property name="visible">True</property> | 168 | <property name="visible">True</property> |
| 230 | 6 | <property name="can_focus">False</property> | 169 | <property name="can_focus">False</property> |
| 231 | @@ -185,118 +348,60 @@ | |||
| 232 | 185 | <property name="height">1</property> | 348 | <property name="height">1</property> |
| 233 | 186 | </packing> | 349 | </packing> |
| 234 | 187 | </child> | 350 | </child> |
| 235 | 351 | |||
| 236 | 188 | <child> | 352 | <child> |
| 238 | 189 | <object class="GtkLabel" id="use_crypto_desc"> | 353 | <object class="GtkBox" id="gtkbox1"> |
| 239 | 190 | <property name="visible">True</property> | 354 | <property name="visible">True</property> |
| 240 | 191 | <property name="can_focus">False</property> | 355 | <property name="can_focus">False</property> |
| 266 | 192 | <property name="margin_left">48</property> | 356 | <child> |
| 267 | 193 | <property name="xalign">0</property> | 357 | <object class="GtkButton" id="advanced_features_button"> |
| 268 | 194 | <property name="yalign">0</property> | 358 | <property name="visible">True</property> |
| 269 | 195 | <property name="label" translatable="yes"><span size="small">You’ll choose a security key in the next step.</span></property> | 359 | <property name="can_focus">True</property> |
| 270 | 196 | <property name="use_markup">True</property> | 360 | <property name="margin_left">24</property> |
| 271 | 197 | <property name="wrap">True</property> | 361 | <property name="receives_default">False</property> |
| 272 | 198 | </object> | 362 | <property name="xalign">0</property> |
| 273 | 199 | <packing> | 363 | <signal name="clicked" handler="on_advanced_features_clicked" swapped="no"/> |
| 274 | 200 | <property name="left_attach">0</property> | 364 | <child> |
| 275 | 201 | <property name="top_attach">9</property> | 365 | <object class="GtkLabel" id="advanced_features_selected"> |
| 276 | 202 | <property name="width">1</property> | 366 | <property name="visible">True</property> |
| 277 | 203 | <property name="height">1</property> | 367 | <property name="can_focus">False</property> |
| 278 | 204 | </packing> | 368 | <property name="label" translatable="yes">Advanced features...</property> |
| 279 | 205 | </child> | 369 | <property name="use_markup">True</property> |
| 280 | 206 | <child> | 370 | </object> |
| 281 | 207 | <object class="GtkCheckButton" id="use_crypto"> | 371 | </child> |
| 282 | 208 | <property name="label" translatable="yes">Encrypt the new Ubuntu installation for security</property> | 372 | </object> |
| 283 | 209 | <property name="visible">True</property> | 373 | <packing> |
| 284 | 210 | <property name="can_focus">True</property> | 374 | <property name="expand">False</property> |
| 285 | 211 | <property name="margin_left">24</property> | 375 | <property name="fill">True</property> |
| 286 | 212 | <property name="receives_default">False</property> | 376 | <property name="position">1</property> |
| 287 | 213 | <property name="margin_top">9</property> | 377 | <property name="left_attach">0</property> |
| 288 | 214 | <property name="xalign">0</property> | 378 | <property name="width">1</property> |
| 289 | 215 | <property name="draw_indicator">True</property> | 379 | <property name="height">1</property> |
| 290 | 216 | <signal name="clicked" handler="on_crypto_lvm_toggled" swapped="no"/> | 380 | </packing> |
| 291 | 381 | </child> | ||
| 292 | 382 | <child> | ||
| 293 | 383 | <object class="GtkLabel" id="advanced_features_desc"> | ||
| 294 | 384 | <property name="visible">True</property> | ||
| 295 | 385 | <property name="can_focus">False</property> | ||
| 296 | 386 | <property name="label" translatable="yes">None selected</property> | ||
| 297 | 387 | <property name="margin_left">9</property> | ||
| 298 | 388 | </object> | ||
| 299 | 389 | <packing> | ||
| 300 | 390 | <property name="expand">False</property> | ||
| 301 | 391 | <property name="fill">True</property> | ||
| 302 | 392 | <property name="position">1</property> | ||
| 303 | 393 | <property name="width">1</property> | ||
| 304 | 394 | <property name="height">1</property> | ||
| 305 | 395 | </packing> | ||
| 306 | 396 | </child> | ||
| 307 | 217 | </object> | 397 | </object> |
| 308 | 218 | <packing> | 398 | <packing> |
| 309 | 399 | <property name="expand">False</property> | ||
| 310 | 400 | <property name="fill">True</property> | ||
| 311 | 401 | <property name="position">2</property> | ||
| 312 | 219 | <property name="left_attach">0</property> | 402 | <property name="left_attach">0</property> |
| 313 | 220 | <property name="top_attach">8</property> | 403 | <property name="top_attach">8</property> |
| 314 | 221 | <property name="width">1</property> | ||
| 315 | 222 | <property name="height">1</property> | ||
| 316 | 223 | </packing> | ||
| 317 | 224 | </child> | ||
| 318 | 225 | <child> | ||
| 319 | 226 | <object class="GtkLabel" id="use_lvm_desc"> | ||
| 320 | 227 | <property name="visible">True</property> | ||
| 321 | 228 | <property name="can_focus">False</property> | ||
| 322 | 229 | <property name="margin_left">48</property> | ||
| 323 | 230 | <property name="xalign">0</property> | ||
| 324 | 231 | <property name="yalign">0</property> | ||
| 325 | 232 | <property name="label" translatable="yes"><span size="small">This will set up Logical Volume Management. It allows taking snapshots and easier partition resizing.</span></property> | ||
| 326 | 233 | <property name="use_markup">True</property> | ||
| 327 | 234 | <property name="wrap">True</property> | ||
| 328 | 235 | </object> | ||
| 329 | 236 | <packing> | ||
| 330 | 237 | <property name="left_attach">0</property> | ||
| 331 | 238 | <property name="top_attach">11</property> | ||
| 332 | 239 | <property name="width">1</property> | ||
| 333 | 240 | <property name="height">1</property> | ||
| 334 | 241 | </packing> | ||
| 335 | 242 | </child> | ||
| 336 | 243 | <child> | ||
| 337 | 244 | <object class="GtkCheckButton" id="use_lvm"> | ||
| 338 | 245 | <property name="label" translatable="yes">Use LVM with the new Ubuntu installation</property> | ||
| 339 | 246 | <property name="visible">True</property> | ||
| 340 | 247 | <property name="can_focus">True</property> | ||
| 341 | 248 | <property name="margin_left">24</property> | ||
| 342 | 249 | <property name="receives_default">False</property> | ||
| 343 | 250 | <property name="margin_top">9</property> | ||
| 344 | 251 | <property name="xalign">0</property> | ||
| 345 | 252 | <property name="draw_indicator">True</property> | ||
| 346 | 253 | <signal name="clicked" handler="on_crypto_lvm_toggled" swapped="no"/> | ||
| 347 | 254 | </object> | ||
| 348 | 255 | <packing> | ||
| 349 | 256 | <property name="left_attach">0</property> | ||
| 350 | 257 | <property name="top_attach">10</property> | ||
| 351 | 258 | <property name="width">1</property> | ||
| 352 | 259 | <property name="height">1</property> | ||
| 353 | 260 | </packing> | ||
| 354 | 261 | </child> | ||
| 355 | 262 | 404 | ||
| 356 | 263 | <child> | ||
| 357 | 264 | <object class="GtkLabel" id="use_zfs_desc"> | ||
| 358 | 265 | <property name="visible">True</property> | ||
| 359 | 266 | <property name="can_focus">False</property> | ||
| 360 | 267 | <property name="margin_left">24</property> | ||
| 361 | 268 | <property name="xalign">0</property> | ||
| 362 | 269 | <property name="yalign">0</property> | ||
| 363 | 270 | <property name="label" translatable="yes"><span foreground="darkred">Warning:</span> This will delete all your files on all operating systems. | ||
| 364 | 271 | This is experimental and may cause data loss. Do not use on production systems.</property> | ||
| 365 | 272 | <property name="use_markup">True</property> | ||
| 366 | 273 | <property name="wrap">True</property> | ||
| 367 | 274 | </object> | ||
| 368 | 275 | <packing> | ||
| 369 | 276 | <property name="left_attach">0</property> | ||
| 370 | 277 | <property name="top_attach">14</property> | ||
| 371 | 278 | <property name="width">1</property> | ||
| 372 | 279 | <property name="height">1</property> | ||
| 373 | 280 | </packing> | ||
| 374 | 281 | </child> | ||
| 375 | 282 | <child> | ||
| 376 | 283 | <object class="GtkRadioButton" id="use_zfs"> | ||
| 377 | 284 | <property name="label" translatable="yes">EXPERIMENTAL: Erase disk and use ZFS</property> | ||
| 378 | 285 | <property name="visible">True</property> | ||
| 379 | 286 | <property name="can_focus">True</property> | ||
| 380 | 287 | <property name="receives_default">False</property> | ||
| 381 | 288 | <property name="margin_top">9</property> | ||
| 382 | 289 | <property name="xalign">0</property> | ||
| 383 | 290 | <property name="active">True</property> | ||
| 384 | 291 | <property name="draw_indicator">True</property> | ||
| 385 | 292 | <property name="group">reuse_partition</property> | ||
| 386 | 293 | <signal name="clicked" handler="part_ask_option_changed" swapped="no"/> | ||
| 387 | 294 | </object> | ||
| 388 | 295 | <packing> | ||
| 389 | 296 | <property name="left_attach">0</property> | ||
| 390 | 297 | <property name="top_attach">13</property> | ||
| 391 | 298 | <property name="width">1</property> | ||
| 392 | 299 | <property name="height">1</property> | ||
| 393 | 300 | </packing> | 405 | </packing> |
| 394 | 301 | </child> | 406 | </child> |
| 395 | 302 | 407 | ||
| 396 | @@ -313,7 +418,7 @@ This is experimental and may cause data loss. Do not use on production systems.< | |||
| 397 | 313 | </object> | 418 | </object> |
| 398 | 314 | <packing> | 419 | <packing> |
| 399 | 315 | <property name="left_attach">0</property> | 420 | <property name="left_attach">0</property> |
| 401 | 316 | <property name="top_attach">16</property> | 421 | <property name="top_attach">10</property> |
| 402 | 317 | <property name="width">1</property> | 422 | <property name="width">1</property> |
| 403 | 318 | <property name="height">1</property> | 423 | <property name="height">1</property> |
| 404 | 319 | </packing> | 424 | </packing> |
| 405 | @@ -333,7 +438,7 @@ This is experimental and may cause data loss. Do not use on production systems.< | |||
| 406 | 333 | </object> | 438 | </object> |
| 407 | 334 | <packing> | 439 | <packing> |
| 408 | 335 | <property name="left_attach">0</property> | 440 | <property name="left_attach">0</property> |
| 410 | 336 | <property name="top_attach">15</property> | 441 | <property name="top_attach">9</property> |
| 411 | 337 | <property name="width">1</property> | 442 | <property name="width">1</property> |
| 412 | 338 | <property name="height">1</property> | 443 | <property name="height">1</property> |
| 413 | 339 | </packing> | 444 | </packing> |
| 414 | diff --git a/tests/run-pep8 b/tests/run-pep8 | |||
| 415 | index c0c8303..a71211d 100755 | |||
| 416 | --- a/tests/run-pep8 | |||
| 417 | +++ b/tests/run-pep8 | |||
| 418 | @@ -5,5 +5,6 @@ | |||
| 419 | 5 | # FIXME: re-enable W504 and update for new breaking rules for binary op | 5 | # FIXME: re-enable W504 and update for new breaking rules for binary op |
| 420 | 6 | pep8 --exclude keyboard_names.py --exclude Keyboard.py \ | 6 | pep8 --exclude keyboard_names.py --exclude Keyboard.py \ |
| 421 | 7 | --ignore=E402,W504 \ | 7 | --ignore=E402,W504 \ |
| 422 | 8 | --max-line-length=119 \ | ||
| 423 | 8 | $(grep -lr '#!.*bin/python' bin) \ | 9 | $(grep -lr '#!.*bin/python' bin) \ |
| 424 | 9 | scripts tests tests/run tests/run-frontend ubiquity autopilot | 10 | scripts tests tests/run tests/run-frontend ubiquity autopilot |
| 425 | diff --git a/tests/test_gtkui.py b/tests/test_gtkui.py | |||
| 426 | index 5a948bf..60cbfd3 100644 | |||
| 427 | --- a/tests/test_gtkui.py | |||
| 428 | +++ b/tests/test_gtkui.py | |||
| 429 | @@ -106,6 +106,8 @@ class TestFrontend(unittest.TestCase): | |||
| 430 | 106 | 'cancelbutton2', 'okbutton2', 'okbutton3', | 106 | 'cancelbutton2', 'okbutton2', 'okbutton3', |
| 431 | 107 | 'partition_dialog_okbutton', 'cancelbutton3', | 107 | 'partition_dialog_okbutton', 'cancelbutton3', |
| 432 | 108 | 'grub_fail_okbutton', | 108 | 'grub_fail_okbutton', |
| 433 | 109 | 'advanced_features_cancelbutton', | ||
| 434 | 110 | 'advanced_features_okbutton', | ||
| 435 | 109 | # These are calculated and set as the partitioning options are | 111 | # These are calculated and set as the partitioning options are |
| 436 | 110 | # being calculated. | 112 | # being calculated. |
| 437 | 111 | 'reuse_partition_desc', 'reuse_partition', | 113 | 'reuse_partition_desc', 'reuse_partition', |
| 438 | @@ -113,6 +115,7 @@ class TestFrontend(unittest.TestCase): | |||
| 439 | 113 | 'resize_use_free_desc', 'resize_use_free', | 115 | 'resize_use_free_desc', 'resize_use_free', |
| 440 | 114 | 'use_device_desc', 'use_device', 'part_ask_heading', | 116 | 'use_device_desc', 'use_device', 'part_ask_heading', |
| 441 | 115 | 'custom_partitioning_desc', 'custom_partitioning', | 117 | 'custom_partitioning_desc', 'custom_partitioning', |
| 442 | 118 | 'advanced_features_selected', | ||
| 443 | 116 | # Pulled straight from debconf when the installation medium is | 119 | # Pulled straight from debconf when the installation medium is |
| 444 | 117 | # already mounted. | 120 | # already mounted. |
| 445 | 118 | 'part_advanced_warning_message', | 121 | 'part_advanced_warning_message', |
| 446 | diff --git a/ubiquity/plugins/ubi-partman.py b/ubiquity/plugins/ubi-partman.py | |||
| 447 | index a1f731d..5601d7d 100644 | |||
| 448 | --- a/ubiquity/plugins/ubi-partman.py | |||
| 449 | +++ b/ubiquity/plugins/ubi-partman.py | |||
| 450 | @@ -200,10 +200,6 @@ class PageGtk(PageBase): | |||
| 451 | 200 | self.partition_toolbar.child_set_property(wdg, 'homogeneous', | 200 | self.partition_toolbar.child_set_property(wdg, 'homogeneous', |
| 452 | 201 | False) | 201 | False) |
| 453 | 202 | 202 | ||
| 454 | 203 | zpool_exists = os.path.exists('/sbin/zpool') | ||
| 455 | 204 | self.use_zfs.set_visible(zpool_exists) | ||
| 456 | 205 | self.use_zfs_desc.set_visible(zpool_exists) | ||
| 457 | 206 | |||
| 458 | 207 | # GtkBuilder signal mapping is broken (LP: # 852054). | 203 | # GtkBuilder signal mapping is broken (LP: # 852054). |
| 459 | 208 | self.part_auto_hidden_label.connect( | 204 | self.part_auto_hidden_label.connect( |
| 460 | 209 | 'activate-link', self.part_auto_hidden_label_activate_link) | 205 | 'activate-link', self.part_auto_hidden_label_activate_link) |
| 461 | @@ -289,6 +285,44 @@ class PageGtk(PageBase): | |||
| 462 | 289 | width, height) | 285 | width, height) |
| 463 | 290 | widget.show() | 286 | widget.show() |
| 464 | 291 | 287 | ||
| 465 | 288 | def on_advanced_features_clicked(self, widget): | ||
| 466 | 289 | from gi.repository import Gtk | ||
| 467 | 290 | |||
| 468 | 291 | # Save current state of advanced features | ||
| 469 | 292 | selected = None | ||
| 470 | 293 | crypto_selected = self.use_crypto.get_active() | ||
| 471 | 294 | for w in (self.advanced_features_radio_none, | ||
| 472 | 295 | self.use_lvm, self.use_zfs): | ||
| 473 | 296 | if w.get_active(): | ||
| 474 | 297 | selected = w | ||
| 475 | 298 | break | ||
| 476 | 299 | |||
| 477 | 300 | # Only show zfs when available | ||
| 478 | 301 | zpool_exists = os.path.exists('/sbin/zpool') | ||
| 479 | 302 | self.use_zfs.set_visible(zpool_exists) | ||
| 480 | 303 | |||
| 481 | 304 | dlg = self.advanced_features_dialog | ||
| 482 | 305 | dlg.show() | ||
| 483 | 306 | response = dlg.run() | ||
| 484 | 307 | dlg.hide() | ||
| 485 | 308 | |||
| 486 | 309 | if response == Gtk.ResponseType.OK: | ||
| 487 | 310 | label = "" | ||
| 488 | 311 | if self.advanced_features_radio_none.get_active(): | ||
| 489 | 312 | label = self.controller.get_string('advanced_features_none_selected') | ||
| 490 | 313 | elif self.use_lvm.get_active(): | ||
| 491 | 314 | label = self.controller.get_string('advanced_features_lvm_selected') | ||
| 492 | 315 | if self.use_crypto.get_active(): | ||
| 493 | 316 | label = self.controller.get_string('advanced_features_crypto_selected') | ||
| 494 | 317 | elif self.use_zfs.get_active(): | ||
| 495 | 318 | label = self.controller.get_string('advanced_features_zfs_selected') | ||
| 496 | 319 | self.advanced_features_desc.set_text(label) | ||
| 497 | 320 | else: | ||
| 498 | 321 | # Restore previous selection | ||
| 499 | 322 | if selected: | ||
| 500 | 323 | selected.set_active(True) | ||
| 501 | 324 | self.use_crypto.set_active(crypto_selected) | ||
| 502 | 325 | |||
| 503 | 292 | def plugin_on_next_clicked(self): | 326 | def plugin_on_next_clicked(self): |
| 504 | 293 | reuse = self.reuse_partition.get_active() | 327 | reuse = self.reuse_partition.get_active() |
| 505 | 294 | replace = self.replace_partition.get_active() | 328 | replace = self.replace_partition.get_active() |
| 506 | @@ -475,12 +509,20 @@ class PageGtk(PageBase): | |||
| 507 | 475 | self.controller.toggle_next_button() | 509 | self.controller.toggle_next_button() |
| 508 | 476 | self.plugin_is_install = about_to_install | 510 | self.plugin_is_install = about_to_install |
| 509 | 477 | 511 | ||
| 511 | 478 | # Supporting crypto and lvm in new installs only for now | 512 | # Advanced features in new installs only for now |
| 512 | 479 | use_device = self.use_device.get_active() | 513 | use_device = self.use_device.get_active() |
| 517 | 480 | self.use_lvm.set_sensitive(use_device) | 514 | self.advanced_features_button.set_sensitive(use_device) |
| 518 | 481 | self.use_crypto.set_sensitive(use_device) | 515 | self.advanced_features_selected.set_sensitive(use_device) |
| 519 | 482 | self.use_lvm_desc.set_sensitive(use_device) | 516 | |
| 520 | 483 | self.use_crypto_desc.set_sensitive(use_device) | 517 | def advanced_features_option_changed(self, widget): |
| 521 | 518 | if not widget.get_active(): | ||
| 522 | 519 | return | ||
| 523 | 520 | |||
| 524 | 521 | use_lvm = self.use_lvm.get_active() | ||
| 525 | 522 | if not use_lvm: | ||
| 526 | 523 | self.use_crypto.set_active(False) | ||
| 527 | 524 | self.use_crypto.set_sensitive(use_lvm) | ||
| 528 | 525 | self.use_crypto_desc.set_sensitive(use_lvm) | ||
| 529 | 484 | 526 | ||
| 530 | 485 | def initialize_resize_mode(self): | 527 | def initialize_resize_mode(self): |
| 531 | 486 | disk_id = self.get_current_disk_partman_id() | 528 | disk_id = self.get_current_disk_partman_id() |
| 532 | @@ -662,21 +704,23 @@ class PageGtk(PageBase): | |||
| 533 | 662 | ticked = False | 704 | ticked = False |
| 534 | 663 | for option, name in option_to_widget: | 705 | for option, name in option_to_widget: |
| 535 | 664 | opt_widget = getattr(self, name) | 706 | opt_widget = getattr(self, name) |
| 537 | 665 | opt_desc = getattr(self, name + '_desc') | 707 | opt_desc = getattr(self, name + '_desc', None) |
| 538 | 666 | 708 | ||
| 539 | 667 | if option in options: | 709 | if option in options: |
| 540 | 668 | opt_widget.show() | 710 | opt_widget.show() |
| 541 | 669 | opt_desc.show() | ||
| 542 | 670 | opt_widget.set_label(options[option].title) | 711 | opt_widget.set_label(options[option].title) |
| 544 | 671 | opt_desc.set_markup(fmt % options[option].desc) | 712 | |
| 545 | 713 | if opt_desc: | ||
| 546 | 714 | opt_desc.show() | ||
| 547 | 715 | opt_desc.set_markup(fmt % options[option].desc) | ||
| 548 | 716 | |||
| 549 | 672 | if not ticked and opt_widget.get_sensitive(): | 717 | if not ticked and opt_widget.get_sensitive(): |
| 550 | 673 | opt_widget.set_active(True) | 718 | opt_widget.set_active(True) |
| 551 | 674 | ticked = True | 719 | ticked = True |
| 552 | 675 | else: | 720 | else: |
| 553 | 676 | opt_widget.hide() | 721 | opt_widget.hide() |
| 557 | 677 | opt_desc.hide() | 722 | if opt_desc: |
| 558 | 678 | 723 | opt_desc.hide() | |
| 556 | 679 | self.use_zfs_desc.set_markup(fmt % self.use_zfs_desc.get_label()) | ||
| 559 | 680 | 724 | ||
| 560 | 681 | # Process the default selection | 725 | # Process the default selection |
| 561 | 682 | self.part_ask_option_changed(None) | 726 | self.part_ask_option_changed(None) |
I have nothing to add on the code changes themselves.
I have a done some tests with current 19.10 image + this branch on top of it. Apart from the selected translations still in english (but I see them in the template below and it makes sense to update the translations in a separate commits as discussed on IRC before the review), I didn't spot any major issues compared to the linked design.
Next/Back works as expected, even if going to the custom screen selection and back then (previous option still selected). However (and I checked that this isn't a regression from that MP), once pressing back from the custom screen, "Entire disk" is selected instead of "Something else". The previous advanced feature selection is still remembered though.
Just a note: there are some missing margins on the Cancel/OK buttons in the dialog compared to the spec.