Page MenuHomePhabricator

ecore_evas: introduce initial selection & dnd support for x.
AcceptedPublic

Authored by bu5hm4n on Sun, Jan 26, 3:36 AM.

Details

Summary

Seats are not implemented, if there is a type mismatch promises are
going to be rejected. Most of this code is copied over from
selection_manager.

Depends on D11194

Diff Detail

Repository
rEFL core/efl
Branch
devs/bu5hm4n/work_cnp
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 16080
bu5hm4n created this revision.Sun, Jan 26, 3:36 AM
bu5hm4n requested review of this revision.Sun, Jan 26, 3:36 AM
bu5hm4n updated this revision to Diff 28519.Mon, Jan 27, 1:56 AM
bu5hm4n edited the summary of this revision. (Show Details)

fixes & more docs

bu5hm4n updated this revision to Diff 28528.Mon, Jan 27, 5:34 AM

api name changes

bu5hm4n updated this revision to Diff 28565.Mon, Jan 27, 11:37 PM

fix leaks

bu5hm4n updated this revision to Diff 28581.Tue, Jan 28, 5:54 AM

correctly emit the events at startup

bu5hm4n updated this revision to Diff 28975.Wed, Feb 12, 5:51 AM

More fixes and progress

zmike requested changes to this revision.Wed, Feb 12, 7:10 AM

commit log needs to specify that this is for ecore-evas.

overall seems ok

src/modules/ecore_evas/engines/x/ecore_evas_x.c
149

selection

Also this hunk should probably go before the bitfields.

3708

This is in reference to character encoding (https://en.wikipedia.org/wiki/Character_encoding) , and its meaning is that the encoding isn't specified by the protocol and thus can be determined by the selection owner.

In terms of meaning, this more or less should mean "any text format", where the important part in the spec is The type would not actually be TEXT; it would be STRING or some other ATOM naming the encoding chosen by the owner. So we expect/send a specific type rather than text in this case.

In other words, text/plain is probably fine here.

3770

As long as we're in ecore-evas, I'd prefer if we could always pass the parameters ordered with this priority to maintain consistency:

  1. ee is always first when passed
  2. ee private data is always first when passed, or second if ee is passed
This revision now requires changes to proceed.Wed, Feb 12, 7:10 AM
bu5hm4n updated this revision to Diff 28992.Wed, Feb 12, 11:36 AM
bu5hm4n retitled this revision from Introduce initial selection & dnd support for x. to ecore_evas: introduce initial selection & dnd support for x..

fixes

bu5hm4n marked 3 inline comments as done.Wed, Feb 12, 11:47 AM
bu5hm4n updated this revision to Diff 29075.Sat, Feb 15, 12:52 PM

fixes & rebase

zmike accepted this revision.Tue, Feb 18, 9:44 AM

Seems good

This revision is now accepted and ready to land.Tue, Feb 18, 9:44 AM
bu5hm4n updated this revision to Diff 29138.Wed, Feb 19, 11:03 AM

updates due to ecore evas changes