[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/sass-mode 7f0df85 26/31: Nicer buffer handling for sass-ou
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/sass-mode 7f0df85 26/31: Nicer buffer handling for sass-output-region |
Date: |
Sun, 29 Aug 2021 11:29:41 -0400 (EDT) |
branch: elpa/sass-mode
commit 7f0df85fd1b90e40e019a0f2e4ea6661169ceb65
Author: Steve Purcell <steve@sanityinc.com>
Commit: Steve Purcell <steve@sanityinc.com>
Nicer buffer handling for sass-output-region
- Ensure error buffer is shown when relevant
- Put that buffer into view-mode
- Only show output buffer when the output command succeeded
---
sass-mode.el | 31 +++++++++++++++++++------------
1 file changed, 19 insertions(+), 12 deletions(-)
diff --git a/sass-mode.el b/sass-mode.el
index 49cf88f..e271907 100644
--- a/sass-mode.el
+++ b/sass-mode.el
@@ -232,18 +232,25 @@ Called from a program, START and END specify the region
to indent."
(let ((output-buffer "*sass-output*")
(errors-buffer "*sass-errors*")
(region-contents (buffer-substring start end)))
- (with-temp-buffer
- (insert region-contents)
- (newline-and-indent)
- (sass--remove-leading-indent)
- (shell-command-on-region (point-min) (point-max) "sass --stdin"
- output-buffer
- nil
- errors-buffer))
- (when (fboundp 'css-mode)
- (with-current-buffer output-buffer
- (css-mode)))
- (switch-to-buffer-other-window output-buffer)))
+ (let ((exit-code
+ (with-temp-buffer
+ (insert region-contents)
+ (newline-and-indent)
+ (sass--remove-leading-indent)
+ (shell-command-on-region (point-min) (point-max) "sass --stdin"
+ output-buffer
+ nil
+ errors-buffer
+ t))))
+
+ (if (zerop exit-code)
+ (progn
+ (when (fboundp 'css-mode)
+ (with-current-buffer output-buffer
+ (css-mode)))
+ (switch-to-buffer-other-window output-buffer))
+ (with-current-buffer errors-buffer
+ (view-mode))))))
(defun sass-output-buffer ()
"Displays the CSS output for entire buffer."
- [nongnu] elpa/sass-mode 3da9040 23/31: Update my name., (continued)
- [nongnu] elpa/sass-mode 3da9040 23/31: Update my name., ELPA Syncer, 2021/08/29
- [nongnu] elpa/sass-mode 26a66e3 21/31: Revert "Don't override the syntax classes for "-" and "_"", ELPA Syncer, 2021/08/29
- [nongnu] elpa/sass-mode 69fc32b 28/31: Adding sass-before-eval-hook., ELPA Syncer, 2021/08/29
- [nongnu] elpa/sass-mode e0c9198 29/31: Merge pull request #24 from dcluna/add-pre-eval-hook-and-options, ELPA Syncer, 2021/08/29
- [nongnu] elpa/sass-mode 247a0d4 31/31: Ensure cl-return has a matching block to return from (see #23), ELPA Syncer, 2021/08/29
- [nongnu] elpa/sass-mode 3938a42 04/31: Bump version to 3.0.15., ELPA Syncer, 2021/08/29
- [nongnu] elpa/sass-mode c0147d2 10/31: Don't indent after "attr: value" lines (fixes #5), ELPA Syncer, 2021/08/29
- [nongnu] elpa/sass-mode 658f745 11/31: Hyphens are also valid in css attribute names, ELPA Syncer, 2021/08/29
- [nongnu] elpa/sass-mode 702909e 25/31: Fix naming of indent cleanup function, ELPA Syncer, 2021/08/29
- [nongnu] elpa/sass-mode 3494c03 17/31: Merge pull request #15 from chee/patch-1, ELPA Syncer, 2021/08/29
- [nongnu] elpa/sass-mode 7f0df85 26/31: Nicer buffer handling for sass-output-region,
ELPA Syncer <=
- [nongnu] elpa/sass-mode d72a55f 27/31: Add sass-command-options variable., ELPA Syncer, 2021/08/29
- [nongnu] elpa/sass-mode 37105f4 30/31: Make sass-command-options a list of strings, ELPA Syncer, 2021/08/29
- [nongnu] elpa/sass-mode fb636bb 24/31: sass-output-region removing trailing indentation., ELPA Syncer, 2021/08/29