Repository: coala/coala

Issue:
Multiple dots irritating in console table
```
|    .    | 
|    .    | 
|    .    | 
|  32|  32|         if not isinstance(file_diff_dict, dict):
|  33|  33|             raise TypeError("file_diff_dict has to be of type dict.")
|  34|  34| 
|  35|  35|         return self.apply(result, original_file_dict, file_diff_dict, **self.get_metadata().create_params_from_section(section))
|    |    | [NORMAL] LineLengthBear:
|    |    | Line is longer than allowed. (129 > 120)
|    |    | The following options are applicable to this result (choose 0 for no action):
```

The dots in the first three lines are irritating. Putting in the line above the code `...` or `[...]` (or even something else).
So it should look for example like:

```
|  31|  31| [...]
|  32|  32|         if not isinstance(file_diff_dict, dict):
|  33|  33|             raise TypeError("file_diff_dict has to be of type dict.")
|  34|  34| 
|  35|  35|         return self.apply(result, original_file_dict, file_diff_dict, **self.get_metadata().create_params_from_section(section))
|    |    | [NORMAL] LineLengthBear:
|    |    | Line is longer than allowed. (129 > 120)
|    |    | The following options are applicable to this result (choose 0 for no action):
```

or maybe put the dots inside the table like this:

```
| ...| ...|
|  32|  32|         if not isinstance(file_diff_dict, dict):
|  33|  33|             raise TypeError("file_diff_dict has to be of type dict.")
|  34|  34| 
|  35|  35|         return self.apply(result, original_file_dict, file_diff_dict, **self.get_metadata().create_params_from_section(section))
|    |    | [NORMAL] LineLengthBear:
|    |    | Line is longer than allowed. (129 > 120)
|    |    | The following options are applicable to this result (choose 0 for no action):
```

Or if that's too complicated just leave them out.

```
|  32|  32|         if not isinstance(file_diff_dict, dict):
|  33|  33|             raise TypeError("file_diff_dict has to be of type dict.")
|  34|  34| 
|  35|  35|         return self.apply(result, original_file_dict, file_diff_dict, **self.get_metadata().create_params_from_section(section))
|    |    | [NORMAL] LineLengthBear:
|    |    | Line is longer than allowed. (129 > 120)
|    |    | The following options are applicable to this result (choose 0 for no action):
```



Discussion:
I like the third version (`| ...| ...|`) the most.


Resolving patch:
diff --git a/bears/misc/KeywordBear.py b/bears/misc/KeywordBear.py
--- a/bears/misc/KeywordBear.py
+++ b/bears/misc/KeywordBear.py
@@ -12,8 +12,8 @@ def run(self,
         """
         Checks the code files for given keywords.
 
-        :param cs_keywords: A list of keywords to search for case sensitively. Usual examples are TODO and FIXME.
-        :param ci_keywords: A list of keywords to search for case insensitively.
+        :param cs_keywords: A list of keywords to search for (case sensitive). Usual examples are TODO and FIXME.
+        :param ci_keywords: A list of keywords to search for (case insensitive).
         """
         results = []
 
diff --git a/coalib/output/ConsoleInteractor.py b/coalib/output/ConsoleInteractor.py
--- a/coalib/output/ConsoleInteractor.py
+++ b/coalib/output/ConsoleInteractor.py
@@ -109,8 +109,8 @@ def _get_action_info(self, action):
 
         return action.name, section
 
-    def _print_segregation(self, n=3):
-        self.print("\n".join(self._format_line(line="", sign=".") for i in range(n)))
+    def _print_segregation(self):
+        self.print(self._format_line(line="", real_nr="...", sign="|", mod_nr="..."))
 
     def _print_lines(self, file_dict, current_line, result_line, result_file):
         """


Test patch:
diff --git a/coalib/tests/output/ConsoleInteractorTest.py b/coalib/tests/output/ConsoleInteractorTest.py
--- a/coalib/tests/output/ConsoleInteractorTest.py
+++ b/coalib/tests/output/ConsoleInteractorTest.py
@@ -142,7 +142,7 @@ def test_print_results(self):
                                                "line 4\n",
                                                "line 5\n"]})
         self.assertEqual("""\n\nproj/white
-|    .    | \n|    .    | \n|    .    | \n|   2|   2| line 2
+| ...| ...| \n|   2|   2| line 2
 |   3|   3| line 3
 |   4|   4| line 4
 |   5|   5| line 5