From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Philip Kaludercic Newsgroups: gmane.emacs.devel Subject: Re: Proposal to add Popper to ELPA Date: Wed, 06 Sep 2023 17:36:21 +0000 Message-ID: <877cp3fbl6.fsf@posteo.net> References: <874jk8ef0k.fsf@gmail.com> <87y1hkcygb.fsf@posteo.net> <87zg20crbh.fsf@gmail.com> Mime-Version: 1.0 Content-Type: text/plain Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="35429"; mail-complaints-to="usenet@ciao.gmane.io" Cc: emacs-devel@gnu.org To: Karthik Chikmagalur Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Wed Sep 06 19:37:30 2023 Return-path: Envelope-to: ged-emacs-devel@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1qdwT3-0008zo-Cf for ged-emacs-devel@m.gmane-mx.org; Wed, 06 Sep 2023 19:37:29 +0200 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1qdwS6-0003Iw-MP; Wed, 06 Sep 2023 13:36:30 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1qdwS5-0003Ih-1m for emacs-devel@gnu.org; Wed, 06 Sep 2023 13:36:29 -0400 Original-Received: from mout01.posteo.de ([185.67.36.65]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1qdwS1-0006zM-LK for emacs-devel@gnu.org; Wed, 06 Sep 2023 13:36:28 -0400 Original-Received: from submission (posteo.de [185.67.36.169]) by mout01.posteo.de (Postfix) with ESMTPS id 35951240027 for ; Wed, 6 Sep 2023 19:36:22 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=posteo.net; s=2017; t=1694021782; bh=+mAJz7yMfX2tVPA4W+sor9WLT/AYNPu/kou1EhSNnco=; h=From:To:Cc:Subject:Autocrypt:Date:Message-ID:MIME-Version:From; b=aXFmbeCsXiBElSPfuD3gisGpGoN+PNN178hxUwkfnYBaSg13voa3/TGgxti1qKwpI JNfW1PySuBbvFeIkzc5dQbVMfa27pVEryToVQum8TWiORiVZdKYsuXY8+4CEAO+xLj HoehczInhSG84DO+MzJdc6iLv6od/vxjGh0F+aZuNoVDfjhqQwUVGyNs0l2HxUiJke DJepU0iwVcxgX/QdzXZO8sOMm42AyXouSMmd8o2u4FPubWDLF2sobt9Krsc2Fdv9J0 /3CBLDHPVhPxvnqxzYteN8nE44+2wqm7w7eZkJlHf6foWHGtNPwvdRnEKH39xmTBm+ 02zPMrNq7V3LQ== Original-Received: from customer (localhost [127.0.0.1]) by submission (posteo.de) with ESMTPSA id 4RgqKx6H4Lz9rxD; Wed, 6 Sep 2023 19:36:21 +0200 (CEST) In-Reply-To: <87zg20crbh.fsf@gmail.com> (Karthik Chikmagalur's message of "Tue, 05 Sep 2023 13:12:34 -0700") Autocrypt: addr=philipk@posteo.net; keydata= mDMEZBBQQhYJKwYBBAHaRw8BAQdAHJuofBrfqFh12uQu0Yi7mrl525F28eTmwUDflFNmdui0QlBo aWxpcCBLYWx1ZGVyY2ljIChnZW5lcmF0ZWQgYnkgYXV0b2NyeXB0LmVsKSA8cGhpbGlwa0Bwb3N0 ZW8ubmV0PoiWBBMWCAA+FiEEDg7HY17ghYlni8XN8xYDWXahwukFAmQQUEICGwMFCQHhM4AFCwkI BwIGFQoJCAsCBBYCAwECHgECF4AACgkQ8xYDWXahwulikAEA77hloUiSrXgFkUVJhlKBpLCHUjA0 mWZ9j9w5d08+jVwBAK6c4iGP7j+/PhbkxaEKa4V3MzIl7zJkcNNjHCXmvFcEuDgEZBBQQhIKKwYB BAGXVQEFAQEHQI5NLiLRjZy3OfSt1dhCmFyn+fN/QKELUYQetiaoe+MMAwEIB4h+BBgWCAAmFiEE Dg7HY17ghYlni8XN8xYDWXahwukFAmQQUEICGwwFCQHhM4AACgkQ8xYDWXahwukm+wEA8cml4JpK NeAu65rg+auKrPOP6TP/4YWRCTIvuYDm0joBALw98AMz7/qMHvSCeU/hw9PL6u6R2EScxtpKnWof z4oM Received-SPF: pass client-ip=185.67.36.65; envelope-from=philipk@posteo.net; helo=mout01.posteo.de X-Spam_score_int: -43 X-Spam_score: -4.4 X-Spam_bar: ---- X-Spam_report: (-4.4 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H5=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.devel:310215 Archived-At: Karthik Chikmagalur writes: > Thank you for the review, especially for the changes to the defcustoms. > I'm not very familiar with the many options the customization interface > provides. I have applied all changes except the one in the defcustom > for `popper-reference-buffers' (which see below). > > In case you missed it during the review, the package has two files: > `popper.el' and `popper-echo.el'. (The latter provides `popper-echo-mode', > which adds echo-area display of available popups and allows for quick > selection using a transient keymap.) > >> +;; If you are interested in depending on Compat, you could use >> +;; `buffer-match-p' here. >> (defcustom popper-reference-buffers '("\\*Messages\\*$") >> "List of buffers to treat as popups. > > I would like to avoid depending on Compat for one utility function. That is understandable, I just wanted to bring it up. >> - :type '(restricted-sexp :match-alternatives (stringp symbolp functionp consp)) >> - :group 'popper) >> + :type '(repeat (string :tag "Regular Expression") >> + (symbol :tag "Major Mode") >> + (function :tag "Predicate Function") >> + ;; What is the consp in (restricted-sexp :match-alternatives (stringp symbolp functionp consp))? >> + )) > > The `consp' here is for specifying additional behavior per popup. > Presently this is limited to requesting that (initial) display of the > popup be suppressed, but support for other special behavior could be > provided in the future. Examples of this are in the README in the > section "Suppressing popups". > > An example: `("\\*Compilation\\*" . hide)' > > How do you suggest incorporating this into the defcustom? > >> Also, it seems adding a .elpaignore file would be nice to remove >> unnecessary files from the tarball: For now a file just containing >> "images" should suffice. > > Done, thank you. > >> Also also, by default the README file will be used to generate a package >> description (as seen in C-h P). I feel that the current file is just a >> tad too long for this indent, and the description in the commentary >> section might be preferable. Would you be fine with using that instead? > > Yes, that should be fine. The commentary section explains the purpose > of the package and the main functions. 1+ >>> What is Popper? >>> >>> Short for "Popup Buffer", >> >> FWIW I don't think I would have understood this. Perhaps it is just me, >> but despite fearing a general discussion about package names, do you >> think renaming the package to something like "popup-buffers" would be >> imaginable. If not, it is fine, just wanted to bring it up /briefly/. > > I took my cue for the name `popper' from Emacs' built-in `winner-mode'. > At 16K+ downloads, Popper is a reasonably well known package and I think > renaming it now will create more confusion than clarity. For the record, I supposed you are referring to MELPA? > The byline for the package is "Summon and dismiss buffers as popups", > which shows up in package listings and should help discoverability. Of course, which is why I am not insisting on anything, I just wanted to point out that people like me with slight dyslexia have difficulties reading and remembering names like these. It was actually just yesterday that I noticed the name is not "poppler". >>> There are other features, and a few video demos at the link. >> >> Is the video mirrored on some other platform as well? > > It's not. Do you have a suggestion for where you'd like the videos to > be available? I can make that happen. I guess any peertube instance should be OK: https://joinpeertube.org/publish-videos. I haven't taken a look at the video, but as long as there is nothing mentioned in the video that is not mentioned elsewhere, everything should be fine. While some people prefer it, others find videos to be a information-sparse format. > Karthik