Merge pull request #342 from erodriguez23777/fix/visual-mode-yank

fix: include last character when yanking in visual mode

Author: darrenburns

.coverage

@@ -1,7 +1,8 @@
 ## [Unreleased]
 
 ### Fixed
-- Fixed the test snapshot for test_select_post_method
+- Fixed themes not auto-reloading when the theme file is a symlink.
+- Fixed last character being omitted when yanking text in visual mode.
 
 ### Changed
 - `MethodSelector`: uppercase letters can now be used to change the method.

docs/CHANGELOG.md

@@ -1364,7 +1364,14 @@ async def watch_themes(self) -> None:
 
         from watchfiles import awatch
 
-        async for changes in awatch(self.settings.theme_directory):
+        paths_to_watch = {self.settings.theme_directory}
+        
+        if self.settings.theme_directory.exists():
+            for p in self.settings.theme_directory.iterdir():
+                if p.is_symlink():
+                    paths_to_watch.add(p.resolve().parent)
+
+        async for changes in awatch(*paths_to_watch):
             for _change_type, file_path in changes:
                 if file_path.endswith((".yml", ".yaml")):
                     try:
@@ -1378,10 +1385,6 @@ async def watch_themes(self) -> None:
                         try:
                             self._watch_theme(theme.name)
                         except Exception as e:
-                            # I don't think we want to notify here, as editors often
-                            # use heuristics to determine whether to save a file. This could
-                            # prove jarring if we pop up a notification without the user
-                            # explicitly saving the file in their editor.
                             log.warning(f"Error refreshing CSS: {e}")
 
     def on_mount(self) -> None:

src/posting/app.py

@@ -60,6 +60,7 @@ class UrlInput(PostingInput):
     BINDINGS = [
         Binding("down", "app.focus_next", "Focus next", show=False),
         Binding("alt+down", "jump_to_path_param", "Jump to Path param", show=False),
+        Binding("ctrl+y", "copy_url", "Copy URL", show=False),
     ]
 
     @dataclass
@@ -132,6 +133,13 @@ def action_jump_to_path_param(self) -> None:
                     self.PathParamJumpRequestedFromUrlInput(name=name, input=self)
                 )
                 break
+    
+    def action_copy_url(self) -> None:
+        """Copy the URL to the clipboard."""
+        url = self.value
+        if url:
+            self.app.copy_to_clipboard(url)
+            self.notify(f"Copied URL to clipboard: {url}")
 
 
 class SendRequestButton(Button, can_focus=False):

src/posting/widgets/request/url_bar.py

@@ -499,7 +499,13 @@ def action_cursor_word_right(self, select: bool = False) -> None:
         return super().action_cursor_word_right(self.visual_mode or select)
 
     def action_copy_to_clipboard(self) -> None:
-        text_to_copy = self.selected_text
+        if self.visual_mode and not self.selection.is_empty:
+            start, end = sorted([self.selection.start, self.selection.end])
+            
+            inclusive_end = (end[0], end[1] + 1)
+            text_to_copy = self.get_text_range(start, inclusive_end)
+        else:
+            text_to_copy = self.selected_text
         if text_to_copy:
             message = f"Copied {len(text_to_copy)} characters."
             title = "Selection copied"