Summary: | [EFL] Paragraph selection API | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Grzegorz Czajkowski <g.czajkowski> | ||||||||||
Component: | WebKit EFL | Assignee: | Nobody <webkit-unassigned> | ||||||||||
Status: | RESOLVED WONTFIX | ||||||||||||
Severity: | Enhancement | CC: | antognolli+webkit, gyuyoung.kim, js45.yang, kenneth, leandro, lucas.de.marchi, mcatanzaro, rakuco, sangseok.lim, tonikitoo | ||||||||||
Priority: | P2 | ||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | All | ||||||||||||
OS: | Linux | ||||||||||||
Attachments: |
|
Description
Grzegorz Czajkowski
2011-06-03 00:44:00 PDT
Created attachment 95865 [details]
patch
Comment on attachment 95865 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=95865&action=review What would be a plausible use case for these features? > Source/WebKit/efl/ewk/ewk_frame.cpp:1641 > +Eina_Bool ewk_frame_points_contained_same_paragraph_check(Evas_Object* o, int x1, int y1, int x2, int y2) How about calling this ewk_frame_points_in_same_paragraph_get()? (In reply to comment #2) > (From update of attachment 95865 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=95865&action=review > > What would be a plausible use case for these features? One of a possible scenario: 1. User starts to select text using WebKit's selection. 2. If user reaches end of current paragraph then whole paragraph is selected as rectangle. 3. UI application shows handles to control paragraph's selection. > > > Source/WebKit/efl/ewk/ewk_frame.cpp:1641 > > +Eina_Bool ewk_frame_points_contained_same_paragraph_check(Evas_Object* o, int x1, int y1, int x2, int y2) > > How about calling this ewk_frame_points_in_same_paragraph_get()? Ok. Created attachment 96069 [details]
updated patch
Function's name has been changed to frame_points_in_same_paragraph_get
LGTM. Created attachment 104987 [details]
updated patch
Just re-base old patch and adjust it to the current changes.
Comment on attachment 104987 [details]
updated patch
r- for now as at least the coding style and the documentation (the [in] and [out] are not needed) need to be adjusted.
The functions themselves look very specific to me, but that can be discussed later (it'd be good to have the opinion of someone who works on editing code, for example).
Created attachment 122619 [details]
updated patch
I adjusted patch to new WebKit-Efl coding style.
Comment on attachment 122619 [details] updated patch View in context: https://bugs.webkit.org/attachment.cgi?id=122619&action=review Looks good to me except for API naming I mentioned. > Source/WebKit/efl/ewk/ewk_frame.h:908 > +EAPI Eina_Bool ewk_frame_points_in_same_paragraph_get(const Evas_Object *o, int x1, int y1, int x2, int y2); I'm not sure whether this API naming is good. Comment on attachment 122619 [details] updated patch Cleared review? from attachment 122619 [details] so that this bug does not appear in http://webkit.org/pending-review. If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again). Closing this bug because the EFL port has been removed from trunk. If you feel this bug applies to a different upstream WebKit port and was closed in error, please either update the title and reopen the bug, or leave a comment to request this. |