elm-config: correct wrong "if" statement #46

Merged

raster merged 1 commits from dimmus/efl:devs/dimmus/elm_config_profile into master 2023-12-19 04:08:49 -08:00

Conversation 1 Commits 1 Files Changed 1 +1 -1

dimmus commented 2023-11-15 06:41:25 -08:00

Contributor

Copy Link

If nothing is selected, then there is nothing to disable.
"If" statement did not work here until now.

To test:

1. Run elementary_config from the terminal.
2. Select the Profile menu item (no clicks on any item).
3. Close the app.
4. Verify the error log messages before and after the patch.

@fixed

If nothing is selected, then there is nothing to disable. "If" statement did not work here until now. To test: 1. Run `elementary_config` from the terminal. 2. Select the `Profile` menu item (no clicks on any item). 3. Close the app. 4. Verify the error log messages before and after the patch. @fixed

dimmus added 1 commit 2023-11-15 06:41:27 -08:00

45d7e64775

elm-config: correct wrong "if" statement ...

If nothing is selected then nothing to disable.
"If" statement did not work here until now.

Test:
1. Run elementary_config from terminal.
2. Select Profile menu item (make no clicks on any item).
4. Close the app.
3. Verify the error log messages before to and following the patch.

@fixed

raster commented 2023-12-19 04:08:38 -08:00

Owner

Copy Link

looks good!

looks good!

raster merged commit cc66b5d011 into master 2023-12-19 04:08:49 -08:00

raster deleted branch devs/dimmus/elm_config_profile 2023-12-19 04:08:52 -08:00

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No Label

Milestone

Clear milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

No Assignees

2 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: enlightenment/efl#46

No description provided.