Web lists-archives.com

Re: Review Request 128665: Nested tags for Baloo Widgets




This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128665/

On August 19th, 2016, 8:52 a.m. UTC, Vishesh Handa wrote:

Could you please attach some screenshots of the before and after?

On August 20th, 2016, 10:57 p.m. UTC, James Smith wrote:

I don't think it's really necessary, the existing indexed tag was erased if a tag higher up was selected, or an expanded first-level indexed tag became unexpanded when erasing an unsaved new tag beyond the last item of the indexed tag.

I cannot visaulize how this looks by just reading at the code. Please add images. Otherwise I need to compile this code and try it out manually.


- Vishesh


On August 19th, 2016, 12:05 a.m. UTC, James Smith wrote:

Review request for Baloo.
By James Smith.

Updated Aug. 19, 2016, 12:05 a.m.

Bugs: 334615
Repository: baloo-widgets

Description

Info pane tags are shortened to their last element, with the full tag shown in each tag's tooltip. The tag selector box allows for creating and selecting nested tags.

Testing

Compile, run

Diffs

  • src/kedittagsdialog.cpp (1778d53)
  • src/kedittagsdialog_p.h (3c4de19)
  • src/tagcheckbox.cpp (8273a7f)

View Diff