Improve the README

Author: kylecarbs

README.md

@@ -3,53 +3,44 @@
   Pulldash
 </h1>
 
-The fastest way to review pull requests that makes massive PRs feel instant.
+Review pull requests in a high-performance UI, driven by keybinds.
 
-- Keybord-driven: navigate, comment, and approve without touching your mouse
-- Performant: giant diffs render smoothly, virtualized rendering keeps you at 60fps even on 10k+ line changes
-- Local or hosted: download the desktop app to avoid sending your credentials anywhere
+> [!WARNING]
+> Pulldash is in alpha. Please report bugs.
 
-## Try It
+[![Example](./docs/screenshots/overview.png)](https://pulldash.com)
 
-Head to [pulldash.com](https://pulldash.com) to explore pull-requests (no auth required).
+## Try
 
-[Download for Desktop](https://github.com/coder/pulldash/releases).
+**Browser**: on [pulldash.com](https://pulldash.com) [no auth required]
 
-## Features
+> [!NOTE]
+> GitHub tokens are stored in your browser, never on the Pulldash server post-authentication.
+
+**Desktop**: download the [latest release](https://github.com/coder/pulldash/releases) available for Linux, macOS, and Windows.
 
-- Fast
 
-  ![]
+## Features
 
 - Customize your PR list with search queries:
 
   ![Filtering PRs](./docs/screenshots/filtering.png)
 
-## Why Not GitHub's Web UI?
-
-- Lack of native PR tracking
+- Use keybinds to add/remove comments, select line ranges, switch files, and submit reviews:
 
-GitHub's PR interface is slow, especially for large PRs.
+  ![Keybinds](./docs/screenshots/keybind-driven.png)
 
-GitHub's PR interface is slow, especially for large PRs.
+- Instantly search across files:
 
-| Issue                   | GitHub Web                    | Pulldash                            |
-| ----------------------- | ----------------------------- | ----------------------------------- |
-| **Large PRs**           | Truncates diffs, loads slowly | ✓ Renders everything, stays smooth  |
-| **Keyboard navigation** | Limited shortcuts             | ✓ Full keyboard-driven workflow     |
-| **Multi-file review**   | Constant page loads           | ✓ Instant tab switching             |
-| **Context switching**   | Browser tabs everywhere       | ✓ Dedicated app, focused experience |
+  ![Search](./docs/screenshots/search.png)
 
-GitHub supports [CORS](https://docs.github.com/en/rest/using-the-rest-api/using-cors-and-jsonp-to-make-cross-origin-requests) on their API, making Pulldash a simple UI.
+## Why not GitHub's UI?
 
-## Keyboard Shortcuts
+GitHub supports [CORS](https://docs.github.com/en/rest/using-the-rest-api/using-cors-and-jsonp-to-make-cross-origin-requests) for their API, making Pulldash a simple UI wrapper with some nicities:
 
-| Action          | Shortcut           |
-| --------------- | ------------------ |
-| Command palette | `⌘K` or `⌘P`       |
-| Switch tabs     | `⌘1` - `⌘9`        |
-| Close tab       | `⌘W`               |
-| Navigate files  | `↑` `↓` in palette |
+- Filtering on repositories you care about.
+- Opening large files in GitHub is slow (Pulldash performs all diff parsing and syntax highlighting in worker threads).
+- GitHub lacks comprehensive keybinds that allow you to be fully keyboard-driven.
 
 ## License
 

docs/screenshots/overview.png

@@ -1383,9 +1383,6 @@ const DiffViewer = memo(function DiffViewer({
     overscan: 100,
     // Add padding at the end so we can scroll the last line to center
     paddingEnd: 300,
-    // Account for KeybindsBar height when calculating visibility for scrollToIndex
-    // This prevents lines from being hidden under the bar when jumping to bottom
-    scrollMargin: 50,
   });
 
   const onDragStart = useCallback(
@@ -1659,6 +1656,27 @@ const DiffViewer = memo(function DiffViewer({
           virtualizer.scrollToIndex(rowIndex, {
             align: "auto",
           });
+
+          // Account for KeybindsBar: if line is near bottom of viewport, scroll a bit more
+          // This prevents lines from being hidden under the bar
+          const scrollEl = parentRef.current;
+          if (scrollEl) {
+            requestAnimationFrame(() => {
+              const item = virtualizer
+                .getVirtualItems()
+                .find((v) => v.index === rowIndex);
+              if (item) {
+                const itemBottom = item.start + item.size;
+                const viewportBottom =
+                  scrollEl.scrollTop + scrollEl.clientHeight;
+                const KEYBINDS_BAR_HEIGHT = 50;
+                // If item is within 50px of viewport bottom, scroll down to give clearance
+                if (itemBottom > viewportBottom - KEYBINDS_BAR_HEIGHT) {
+                  scrollEl.scrollTop += KEYBINDS_BAR_HEIGHT;
+                }
+              }
+            });
+          }
         }
       }
     });