denexter created this revision.
denexter added a reviewer: pvuorela.
Herald added a project: Calligra: 3.0.
Herald added a subscriber: Calligra-Devel-list.
denexter requested review of this revision.
REVISION SUMMARY
Include the conditions default style when comparing equality and calculating
t
denexter created this revision.
denexter added a reviewer: pvuorela.
Herald added a project: Calligra: 3.0.
Herald added a subscriber: Calligra-Devel-list.
denexter requested review of this revision.
REVISION SUMMARY
These will have the high bit set which will make them appear to be
non-printab
denexter created this revision.
denexter added a reviewer: pvuorela.
Herald added a project: Calligra: 3.0.
Herald added a subscriber: Calligra-Devel-list.
denexter requested review of this revision.
REVISION SUMMARY
The specification says it and the flags entry must exist if the object type
is
denexter created this revision.
denexter added a reviewer: pvuorela.
Herald added a project: Calligra: 3.0.
Herald added a subscriber: Calligra-Devel-list.
denexter requested review of this revision.
REVISION SUMMARY
Read the size of a single cell anchored drawing from the xdr:ext element. The
denexter created this revision.
denexter added a reviewer: pvuorela.
Herald added a project: Calligra: 3.0.
Herald added a subscriber: Calligra-Devel-list.
denexter requested review of this revision.
REVISION SUMMARY
If an object element has child elements but no shape plugin can be resolved
fo
denexter added inline comments.
INLINE COMMENTS
> pvuorela wrote in PictDestination.cpp:54
> Was Andrew's part, but I think file itself should contain the geometry and
> goals are handled here as optional, i.e. if not set will be read from the
> file.
I think picw and pich are source sizes an
dcaliste added a comment.
@davidllewellynjones, I hadn't time yet to test the patch, but the idea and
implementation is already looking great in my opinion. Maybe, just one remark
after looking at the proposed additions: not saving the formale, but the values
should not be named read-only. I
pvuorela updated this revision to Diff 6.
pvuorela added a comment.
Whitespace adjustments
REPOSITORY
R8 Calligra
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D24943?vs=68746&id=6
BRANCH
unicode
REVISION DETAIL
https://phabricator.kde.org/D24943
AFFECTED FILES
fi
davidllewellynjones created this revision.
davidllewellynjones added reviewers: Calligra: 3.0, pvuorela, dcaliste.
davidllewellynjones added a project: Calligra: 3.0.
Herald added a subscriber: Calligra-Devel-list.
davidllewellynjones requested review of this revision.
REVISION SUMMARY
Loading l
pvuorela added inline comments.
INLINE COMMENTS
> davidllewellynjones wrote in DocumentDestination.cpp:125
> A minor issue, but the spacing around the brackets here doesn't match the
> surrounding code.
Sure.
> davidllewellynjones wrote in PictDestination.cpp:46
> The spec mentions `\wbitmap`
davidllewellynjones accepted this revision.
davidllewellynjones added a comment.
This revision is now accepted and ready to land.
I've made some suggestions and have a couple of queries, but they're all very
minor.
INLINE COMMENTS
> DocumentDestination.cpp:125
> + m_charactersToSkip
This revision was automatically updated to reflect the committed changes.
Closed by commit R8:b17f993c3b20: Migrate csv plugins from kdebug to qdebug
(authored by pvuorela).
REPOSITORY
R8 Calligra
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D24831?vs=68438&id=68884
REVISION DETAIL
dcaliste accepted this revision.
dcaliste added a comment.
This revision is now accepted and ready to land.
The patch looks fine, thank you.
REPOSITORY
R8 Calligra
BRANCH
csv_qdebug (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D24831
To: pvuorela, dcaliste, leini
dcaliste added a comment.
Well, after reading the difference between the logical and physical DPI
values:
https://stackoverflow.com/questions/16561879/what-is-the-difference-between-logicaldpix-and-physicaldpix-in-qt
(see the first answer), using the logical value is consistent with the hard
pvuorela accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R8 Calligra
REVISION DETAIL
https://phabricator.kde.org/D24761
To: davidllewellynjones, #calligra:_3.0, pvuorela
Cc: davidllewellynjones, Calligra-Devel-list, dcaliste, cochise, vandenoever
This revision was not accepted when it landed; it landed in state "Needs
Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit R8:0e9d73d8ce1e: Remove unsupported attributes from
QGraphicsWidgets (authored by pvuorela).
REPOSITORY
R8 Calligra
CHAN
16 matches
Mail list logo