Project

General

Profile

Feature #12247

Feature #5464: Revamp Greeter interface

Feature #8230: Greeter revamp: Phase 1

Remove lists of if clauses

Added by alant over 2 years ago. Updated over 2 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
02/17/2017
Due date:
% Done:

100%

Feature Branch:
greeter:feature/12247-replace-lists-of-if
Type of work:
Code
Blueprint:
Starter:
Affected tool:
Greeter

Description

The greeter code has several ugly lists of if clauses:

        if row == self.listboxrow_text:
            tailsgreeter.utils.popover_toggle(self.text_setting.popover)
        elif row == self.listboxrow_keyboard:
            tailsgreeter.utils.popover_toggle(self.keyboard_setting.popover)
        elif row == self.listboxrow_formats:
            tailsgreeter.utils.popover_toggle(self.format_setting.popover)
        elif row == self.listboxrow_tz:
            tailsgreeter.utils.popover_toggle(self.timezone_setting.popover)

They should be replaced by some metaprogramming, making it easier to add new settings.

History

#1 Updated by alant over 2 years ago

  • Subject changed from Use metaprogrammation to replace lists of if clauses to Remove lists of if clauses

#2 Updated by alant over 2 years ago

  • Assignee changed from alant to intrigeri
  • QA Check set to Ready for QA
  • Feature Branch set to greeter:feature/12247-replace-lists-of-if

#3 Updated by alant over 2 years ago

  • Status changed from New to In Progress

#4 Updated by intrigeri over 2 years ago

  • Target version set to Tails_3.0

#5 Updated by intrigeri over 2 years ago

  • Description updated (diff)

#6 Updated by intrigeri over 2 years ago

  • Assignee changed from intrigeri to alant
  • % Done changed from 0 to 30
  • QA Check changed from Ready for QA to Dev Needed

conflicts with current feature/stretch

#7 Updated by intrigeri over 2 years ago

  • Priority changed from Low to Normal

(3.0 blockers are now Elevated or higher)

#8 Updated by alant over 2 years ago

  • Assignee changed from alant to intrigeri
  • % Done changed from 30 to 50
  • QA Check changed from Dev Needed to Ready for QA

conflicts with current feature/stretch

It now applies on feature/stretch, and works when installing the .deb in a booted ISO.

#9 Updated by intrigeri over 2 years ago

  • Assignee changed from intrigeri to alant
  • QA Check changed from Ready for QA to Dev Needed

alant wrote:

conflicts with current feature/stretch

It now applies on feature/stretch

You probably forgot to push your conflict resolution.

#10 Updated by intrigeri over 2 years ago

  • Status changed from In Progress to Resolved
  • Assignee deleted (alant)
  • % Done changed from 50 to 100
  • QA Check changed from Dev Needed to Pass

Released in 1.0.0~beta6. Thanks for caring about the code sanity!

#11 Updated by intrigeri over 2 years ago

  • Parent task changed from #11643 to #8230

Also available in: Atom PDF