N°8955 add UI block for uninstallation#797
N°8955 add UI block for uninstallation#797Timmy38 wants to merge 20 commits intofeature/uninstallationfrom
Conversation
| justify-content: space-between; | ||
| align-items: center; | ||
|
|
||
| width:450px; |
There was a problem hiding this comment.
Put this in a variable,
also consider if this will be it's final width or if the parent grid will decide its element width
|
|
||
| .ibo-extension-details--actions > button{ | ||
| position:relative; | ||
| top:-3px; |
There was a problem hiding this comment.
What's the point of the top: -3px ?
If we need to position flex element we can probably use justify-* or align-*
| 'force_uninstall', | ||
| 'Force uninstall', | ||
| <<<JS | ||
| this.style.display = 'none'; |
There was a problem hiding this comment.
This can probably be done with a complicated CSS rule, but it would be easier if you need to re-display this element if the input toggler is then re-toggled
This is not mandatory as of today but keep it in mind if you need this behavior
| 'Force uninstall', | ||
| <<<JS | ||
| this.style.display = 'none'; | ||
| this.closest('.ibo-extension-details').querySelector('input[type=checkbox]').disabled = false |
There was a problem hiding this comment.
Do we leave the input disabled here ?
| use Dict; | ||
| use utils; | ||
|
|
||
| class ExtensionDetailsFactory extends AbstractUIBlockFactory |
There was a problem hiding this comment.
UIBlocks factories tend to be named <BlockName>UIBlockFactory
| $oBadgeInstalled = BadgeUIBlockFactory::MakeGreen('installed'); | ||
| $oBadgeInstalled->AddCSSClass('checked'); | ||
| $aBadges[] = $oBadgeInstalled; | ||
| $oBadgeToBeUninstalled = BadgeUIBlockFactory::MakeRed('to be uninstalled'); |
There was a problem hiding this comment.
Missing dict entries, try to add them now before forgetting 😁
Check dictionaries/ui/layouts directory
| @@ -0,0 +1 @@ | |||
|
|
|||
There was a problem hiding this comment.
If you don't plan to use this file for ExtensionDetail behavior you can delete it
Otherwise it can be kept
|
@Timmy38 regarding the formatting of the SCSS, in PHPStorm hit |
CombodoApplicationsAccount
force-pushed
the
feature/8955_add_UIBlock_for_uninstallation
branch
from
46451a3 to
6dc8cef
Compare
|
Discussed with Timothée IRL, we stay focused on the setup wizard for now, integration of the UIBlocks will be done afterwards. |
…t - skip if no model available to make setup work
N°8760 - be able to list modules based on extension choices refactoring: move some classes in a moduleinstallation folder (coming namespace) N°8760 - module dependency check applied before audit N°8760 - make dependency check work during audit N°8760 - fix ci N°8760 - fix ci N°8760 - add GetCreatedIn to get module name based on DBObject class - everything stored in MetaModel during compilation and autoload N°8760 - be able to describe from which module a datamodel class comes via MetaModel created_in field N°8760 - rename GetCreatedIn <- GetModuleName + compute module name live instead having complex stuff in MetaModel/compilation temp review 1 review: renaming InstallationChoicesToModuleConverter review: renaming InstallationChoicesToModuleConverter review: ModuleDiscovery:GetModulesOrderedByDependencies replacing deprecated GetAvailableModules method ci: fix typo cleanup review: rework InstallationChoicesToModuleConverter N°8760 - review tests
N°8864 Fix unneeded char
5 participants
No description provided.