Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

DEL: remove senseless text for separator and nothing#830

Open
avmaksimov wants to merge 1 commit intodoublecmd:masterdoublecmd/doublecmd:masterfrom
avmaksimov:hotdirlist-remove-senseless-textavmaksimov/doublecmd:hotdirlist-remove-senseless-textCopy head branch name to clipboard
Open

DEL: remove senseless text for separator and nothing#830
avmaksimov wants to merge 1 commit intodoublecmd:masterdoublecmd/doublecmd:masterfrom
avmaksimov:hotdirlist-remove-senseless-textavmaksimov/doublecmd:hotdirlist-remove-senseless-textCopy head branch name to clipboard

Conversation

@avmaksimov
Copy link
Contributor

Also "Nothing..." was not translated and text for separator and "nothing" was without a label so it looked strange.

Because of this I removed translated string because there is no place where it can be used.

Well, I would be do some cleaning with this module too. Do you mind if I:

  1. replace "case of" with one condition to if;
  2. remove all concerning hd_COMMAND. I think before it was like a simple for adding command but now this code is unused, Or may be you have a plan for adding command to hotdir list?

Also "Nothing..." was not translated and text for separator and "nothing" was without a label so it looked strange.
@avmaksimov avmaksimov marked this pull request as ready for review February 21, 2023 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.