----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115751/#review49796 -----------------------------------------------------------
Ship it! looks fine to me (would like a +1 from okular people too tough) - Marco Martin On Feb. 14, 2014, 7:10 p.m., Vishesh Handa wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115751/ > ----------------------------------------------------------- > > (Updated Feb. 14, 2014, 7:10 p.m.) > > > Review request for Okular and Marco Martin. > > > Repository: okular > > > Description > ------- > > QML PageItem: Handle document path changes > > PageItem works quite well if the DocumentItem has the path on > construction, but it does not update if the path is initially blank and > then later changed. > > > Diffs > ----- > > active/components/pageitem.h b4fe828 > active/components/pageitem.cpp ea5ce10 > > Diff: https://git.reviewboard.kde.org/r/115751/diff/ > > > Testing > ------- > > Yup. It now works when the path is provided after the DocumentItem's > construction. > > > Thanks, > > Vishesh Handa > >
_______________________________________________ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel