all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [ELPA] New package: vertico-posframe
@ 2021-10-25  3:06 tumashu
  2021-10-25 16:13 ` Philip Kaludercic
  0 siblings, 1 reply; 11+ messages in thread
From: tumashu @ 2021-10-25  3:06 UTC (permalink / raw)
  To: emacs-devel@gnu.org

** What is vertico-posframe
vertico-posframe is an vertico extension, which lets vertico use posframe to show its candidate menu.
NOTE: vertico-posframe requires Emacs 26 and do not support mouse click.

** How to enable vertico-posframe
  (require 'vertico-posframe)
  (vertico-posframe-mode 1)

URL: https://github.com/tumashu/vertico-posframe


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [ELPA] New package: vertico-posframe
  2021-10-25  3:06 [ELPA] New package: vertico-posframe tumashu
@ 2021-10-25 16:13 ` Philip Kaludercic
  2021-10-26  0:45   ` tumashu
  0 siblings, 1 reply; 11+ messages in thread
From: Philip Kaludercic @ 2021-10-25 16:13 UTC (permalink / raw)
  To: tumashu; +Cc: emacs-devel@gnu.org

tumashu <tumashu@163.com> writes:

> ** What is vertico-posframe
> vertico-posframe is an vertico extension, which lets vertico use posframe to show its candidate menu.
> NOTE: vertico-posframe requires Emacs 26 and do not support mouse click.
>
> ** How to enable vertico-posframe
>   (require 'vertico-posframe)
>   (vertico-posframe-mode 1)
>
> URL: https://github.com/tumashu/vertico-posframe

Would it make sense to bundle this directly in Vertico?

-- 
	Philip Kaludercic



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re:Re: [ELPA] New package: vertico-posframe
  2021-10-25 16:13 ` Philip Kaludercic
@ 2021-10-26  0:45   ` tumashu
  2021-10-26  7:25     ` Iñigo Serna
  2021-10-26  9:12     ` Philip Kaludercic
  0 siblings, 2 replies; 11+ messages in thread
From: tumashu @ 2021-10-26  0:45 UTC (permalink / raw)
  To: Philip Kaludercic; +Cc: mail, emacs-devel@gnu.org

















At 2021-10-26 00:13:16, "Philip Kaludercic" <philipk@posteo.net> wrote:
>tumashu <tumashu@163.com> writes:
>
>> ** What is vertico-posframe
>> vertico-posframe is an vertico extension, which lets vertico use posframe to show its candidate menu.
>> NOTE: vertico-posframe requires Emacs 26 and do not support mouse click.
>>
>> ** How to enable vertico-posframe
>>   (require 'vertico-posframe)
>>   (vertico-posframe-mode 1)
>>
>> URL: https://github.com/tumashu/vertico-posframe
>
>Would it make sense to bundle this directly in Vertico?

I think vertico prefer keep  simple and less code, so we should see the opinion of Daniel Mendler 

>
>-- 
>	Philip Kaludercic

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [ELPA] New package: vertico-posframe
  2021-10-26  0:45   ` tumashu
@ 2021-10-26  7:25     ` Iñigo Serna
  2021-10-26  7:35       ` Daniel Mendler
  2021-10-26  9:12     ` Philip Kaludercic
  1 sibling, 1 reply; 11+ messages in thread
From: Iñigo Serna @ 2021-10-26  7:25 UTC (permalink / raw)
  To: tumashu; +Cc: mail, Philip Kaludercic, emacs-devel

Hi,

On 26 October 2021 at 02:45 +02, tumashu  <tumashu@163.com> wrote:

> At 2021-10-26 00:13:16, "Philip Kaludercic" <philipk@posteo.net> 
> wrote:
>>> ** What is vertico-posframe
>>
>>Would it make sense to bundle this directly in Vertico?
>
> I think vertico prefer keep  simple and less code, so we should 
> see the opinion of Daniel Mendler 

It would also add an unneeded dependency of an extenal package not 
used
by most vertico users.
Daniel prefered that these extensions be in their own packages and 
keep
core clean.

-- 
Iñigo Serna



On 26 October 2021 at 02:45 +02, tumashu  <tumashu@163.com> wrote:

> At 2021-10-26 00:13:16, "Philip Kaludercic" <philipk@posteo.net> 
> wrote:
>>tumashu <tumashu@163.com> writes:
>>
>>> ** What is vertico-posframe
>>> vertico-posframe is an vertico extension, which lets vertico 
>>> use posframe to show its candidate menu.
>>> NOTE: vertico-posframe requires Emacs 26 and do not support 
>>> mouse click.
>>>
>>> ** How to enable vertico-posframe
>>>   (require 'vertico-posframe)
>>>   (vertico-posframe-mode 1)
>>>
>>> URL: https://github.com/tumashu/vertico-posframe
>>
>>Would it make sense to bundle this directly in Vertico?
>
> I think vertico prefer keep  simple and less code, so we should 
> see the opinion of Daniel Mendler 
>
>>
>>-- 
>>	Philip Kaludercic



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [ELPA] New package: vertico-posframe
  2021-10-26  7:25     ` Iñigo Serna
@ 2021-10-26  7:35       ` Daniel Mendler
  2021-10-26 12:00         ` Stefan Monnier
  0 siblings, 1 reply; 11+ messages in thread
From: Daniel Mendler @ 2021-10-26  7:35 UTC (permalink / raw)
  To: Iñigo Serna, tumashu; +Cc: Philip Kaludercic, emacs-devel

On 10/26/21 9:25 AM, Iñigo Serna wrote:
> Hi,
> 
> On 26 October 2021 at 02:45 +02, tumashu  <tumashu@163.com> wrote:
> 
>> At 2021-10-26 00:13:16, "Philip Kaludercic" <philipk@posteo.net> 
>> wrote:
>>>> ** What is vertico-posframe
>>>
>>> Would it make sense to bundle this directly in Vertico?
>>
>> I think vertico prefer keep  simple and less code, so we should 
>> see the opinion of Daniel Mendler 
> 
> It would also add an unneeded dependency of an extenal package not 
> used
> by most vertico users.
> Daniel prefered that these extensions be in their own packages and 
> keep
> core clean.

Indeed, the packages should not be bundled together. vertico-posframe is
an independent component, which pulls in a third-party dependency.

Daniel



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [ELPA] New package: vertico-posframe
  2021-10-26  0:45   ` tumashu
  2021-10-26  7:25     ` Iñigo Serna
@ 2021-10-26  9:12     ` Philip Kaludercic
  2021-10-27 18:00       ` Philip Kaludercic
  1 sibling, 1 reply; 11+ messages in thread
From: Philip Kaludercic @ 2021-10-26  9:12 UTC (permalink / raw)
  To: tumashu; +Cc: mail, emacs-devel@gnu.org

tumashu  <tumashu@163.com> writes:

> At 2021-10-26 00:13:16, "Philip Kaludercic" <philipk@posteo.net> wrote:
>>tumashu <tumashu@163.com> writes:
>>
>>> ** What is vertico-posframe
>>> vertico-posframe is an vertico extension, which lets vertico use posframe to show its candidate menu.
>>> NOTE: vertico-posframe requires Emacs 26 and do not support mouse click.
>>>
>>> ** How to enable vertico-posframe
>>>   (require 'vertico-posframe)
>>>   (vertico-posframe-mode 1)
>>>
>>> URL: https://github.com/tumashu/vertico-posframe
>>
>>Would it make sense to bundle this directly in Vertico?
>
> I think vertico prefer keep  simple and less code, so we should see the opinion of Daniel Mendler 

No problem, then it should be possible to add your package too.  I will
look over the code later today to check if there is anything that might
be worth discussing.

-- 
	Philip Kaludercic



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [ELPA] New package: vertico-posframe
  2021-10-26  7:35       ` Daniel Mendler
@ 2021-10-26 12:00         ` Stefan Monnier
  2021-10-26 16:51           ` Philip Kaludercic
  0 siblings, 1 reply; 11+ messages in thread
From: Stefan Monnier @ 2021-10-26 12:00 UTC (permalink / raw)
  To: Daniel Mendler; +Cc: Iñigo Serna, tumashu, Philip Kaludercic, emacs-devel

> Indeed, the packages should not be bundled together.  vertico-posframe is
> an independent component, which pulls in a third-party dependency.

I don't have an opinion on how those should be packaged, but I'll just
point out that Vertico would not needed to have `posframe` in its
`Package-Requires` in order to provide support for Posframe.


        Stefan




^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [ELPA] New package: vertico-posframe
  2021-10-26 12:00         ` Stefan Monnier
@ 2021-10-26 16:51           ` Philip Kaludercic
  2021-10-26 18:19             ` Stefan Monnier
  0 siblings, 1 reply; 11+ messages in thread
From: Philip Kaludercic @ 2021-10-26 16:51 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Daniel Mendler, emacs-devel, Iñigo Serna, tumashu

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>> Indeed, the packages should not be bundled together.  vertico-posframe is
>> an independent component, which pulls in a third-party dependency.
>
> I don't have an opinion on how those should be packaged, but I'll just
> point out that Vertico would not needed to have `posframe` in its
> `Package-Requires` in order to provide support for Posframe.

Yes, this was what I was thinking about. A soft dependency that would
raise an error of the posframe part were actually required, but would
otherwise ignore it.

But as Daniel has said he prefers not to do it this way, we can add
vertico-posframe as a separate package.

-- 
	Philip Kaludercic



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [ELPA] New package: vertico-posframe
  2021-10-26 16:51           ` Philip Kaludercic
@ 2021-10-26 18:19             ` Stefan Monnier
  0 siblings, 0 replies; 11+ messages in thread
From: Stefan Monnier @ 2021-10-26 18:19 UTC (permalink / raw)
  To: Philip Kaludercic; +Cc: Daniel Mendler, Iñigo Serna, tumashu, emacs-devel

Philip Kaludercic [2021-10-26 16:51:10] wrote:
> Stefan Monnier <monnier@iro.umontreal.ca> writes:
>>> Indeed, the packages should not be bundled together.  vertico-posframe is
>>> an independent component, which pulls in a third-party dependency.
>>
>> I don't have an opinion on how those should be packaged, but I'll just
>> point out that Vertico would not needed to have `posframe` in its
>> `Package-Requires` in order to provide support for Posframe.
>
> Yes, this was what I was thinking about. A soft dependency that would
> raise an error of the posframe part were actually required, but would
> otherwise ignore it.
>
> But as Daniel has said he prefers not to do it this way, we can add
> vertico-posframe as a separate package.

Yes, either way is fine.


        Stefan




^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [ELPA] New package: vertico-posframe
  2021-10-26  9:12     ` Philip Kaludercic
@ 2021-10-27 18:00       ` Philip Kaludercic
  2021-10-28  9:12         ` tumashu
  0 siblings, 1 reply; 11+ messages in thread
From: Philip Kaludercic @ 2021-10-27 18:00 UTC (permalink / raw)
  To: tumashu; +Cc: mail, emacs-devel@gnu.org

Philip Kaludercic <philipk@posteo.net> writes:

> tumashu  <tumashu@163.com> writes:
>
>> At 2021-10-26 00:13:16, "Philip Kaludercic" <philipk@posteo.net> wrote:
>>>tumashu <tumashu@163.com> writes:
>>>
>>>> ** What is vertico-posframe
>>>> vertico-posframe is an vertico extension, which lets vertico use posframe to show its candidate menu.
>>>> NOTE: vertico-posframe requires Emacs 26 and do not support mouse click.
>>>>
>>>> ** How to enable vertico-posframe
>>>>   (require 'vertico-posframe)
>>>>   (vertico-posframe-mode 1)
>>>>
>>>> URL: https://github.com/tumashu/vertico-posframe
>>>
>>>Would it make sense to bundle this directly in Vertico?
>>
>> I think vertico prefer keep  simple and less code, so we should see the opinion of Daniel Mendler 
>
> No problem, then it should be possible to add your package too.  I will
> look over the code later today to check if there is anything that might
> be worth discussing.

The package builds as expected so I'll add it with auto-syncing as there
haven't been any objections.  I'd just question why you are requiring
cl-lib? It seems you aren't using it?

-- 
	Philip Kaludercic



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re:Re: [ELPA] New package: vertico-posframe
  2021-10-27 18:00       ` Philip Kaludercic
@ 2021-10-28  9:12         ` tumashu
  0 siblings, 0 replies; 11+ messages in thread
From: tumashu @ 2021-10-28  9:12 UTC (permalink / raw)
  To: Philip Kaludercic; +Cc: Daniel Mendler, emacs-devel@gnu.org

















At 2021-10-28 02:00:23, "Philip Kaludercic" <philipk@posteo.net> wrote:
>Philip Kaludercic <philipk@posteo.net> writes:
>
>> tumashu  <tumashu@163.com> writes:
>>
>>> At 2021-10-26 00:13:16, "Philip Kaludercic" <philipk@posteo.net> wrote:
>>>>tumashu <tumashu@163.com> writes:
>>>>
>>>>> ** What is vertico-posframe
>>>>> vertico-posframe is an vertico extension, which lets vertico use posframe to show its candidate menu.
>>>>> NOTE: vertico-posframe requires Emacs 26 and do not support mouse click.
>>>>>
>>>>> ** How to enable vertico-posframe
>>>>>   (require 'vertico-posframe)
>>>>>   (vertico-posframe-mode 1)
>>>>>
>>>>> URL: https://github.com/tumashu/vertico-posframe
>>>>
>>>>Would it make sense to bundle this directly in Vertico?
>>>
>>> I think vertico prefer keep  simple and less code, so we should see the opinion of Daniel Mendler 
>>
>> No problem, then it should be possible to add your package too.  I will
>> look over the code later today to check if there is anything that might
>> be worth discussing.
>
>The package builds as expected so I'll add it with auto-syncing as there
>haven't been any objections.  I'd just question why you are requiring
>cl-lib? It seems you aren't using it?

Removed, thanks!
>
>-- 
>	Philip Kaludercic

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2021-10-28  9:12 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-10-25  3:06 [ELPA] New package: vertico-posframe tumashu
2021-10-25 16:13 ` Philip Kaludercic
2021-10-26  0:45   ` tumashu
2021-10-26  7:25     ` Iñigo Serna
2021-10-26  7:35       ` Daniel Mendler
2021-10-26 12:00         ` Stefan Monnier
2021-10-26 16:51           ` Philip Kaludercic
2021-10-26 18:19             ` Stefan Monnier
2021-10-26  9:12     ` Philip Kaludercic
2021-10-27 18:00       ` Philip Kaludercic
2021-10-28  9:12         ` tumashu

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.