Review Request 110949: Renaming "Page Borders" to "Page Markers" in Calligra Sheets

Inge Wallin inge at lysator.liu.se
Wed Jul 3 10:10:27 BST 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110949/#review35493
-----------------------------------------------------------

Ship it!


Looks good now, please merge and push.


filters/sheets/libkspreadexport/KSpreadLeader.cc
<http://git.reviewboard.kde.org/r/110949/#comment26035>

    Maybe this property should also be renamed but that's for another patch.



sheets/part/View.cpp
<http://git.reviewboard.kde.org/r/110949/#comment26037>

    page boundaries, not borders


- Inge Wallin


On July 3, 2013, 2:09 a.m., Jigar Raisinghani wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110949/
> -----------------------------------------------------------
> 
> (Updated July 3, 2013, 2:09 a.m.)
> 
> 
> Review request for Calligra, Inge Wallin and Marijn Kruisselbrink.
> 
> 
> Description
> -------
> 
> Currently, Calligra Sheets has a feature called Page Borders which displays the boundaries for pages. This is useful to get an idea of where a page ends and new page begins. But these are NOT actual Borders/Boundaries but just the Markers. Page Borders would mean ACTUAL Borders(customizable) and not just markers. So, Renaming "Page Borders" to "Page Markers" 
> 
> 
> Diffs
> -----
> 
>   filters/sheets/excel/import/ExcelImport.cpp 53decbd 
>   filters/sheets/excel/import/excelimporttoods.cc 42586ea 
>   filters/sheets/libkspreadexport/KSpreadLeader.cc 9880b11 
>   sheets/ApplicationSettings.h c3828f2 
>   sheets/ApplicationSettings.cpp c779129 
>   sheets/Sheet.h 7090a8c 
>   sheets/Sheet.cpp 7d3548a 
>   sheets/commands/PageBreakCommand.cpp 7f0f49a 
>   sheets/commands/SheetCommands.h 68a30e1 
>   sheets/commands/SheetCommands.cpp b501c06 
>   sheets/interfaces/SheetAdaptor.h cf0ec69 
>   sheets/interfaces/SheetAdaptor.cpp c4a51ce 
>   sheets/part/View.h 4be69c4 
>   sheets/part/View.cpp 61c485c 
>   sheets/part/dialogs/InterfaceOptionsWidget.ui 0afc27c 
>   sheets/part/dialogs/PreferenceDialog.cpp 721ce25 
>   sheets/part/dialogs/SheetPropertiesDialog.h 2330ae6 
>   sheets/part/dialogs/SheetPropertiesDialog.cpp 858fdb3 
>   sheets/part/dialogs/SheetPropertiesWidget.ui 1d41fc7 
>   sheets/sheets.kcfg c0ec520 
>   sheets/sheets.rc 6256df6 
>   sheets/sheets_readonly.rc 1e9e248 
>   sheets/ui/CellView.cpp d2b6e26 
> 
> Diff: http://git.reviewboard.kde.org/r/110949/diff/
> 
> 
> Testing
> -------
> 
> Yes, Page Borders renamed to Page Markers successfully.
> 
> 
> Thanks,
> 
> Jigar Raisinghani
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20130703/3b6afd02/attachment.htm>


More information about the calligra-devel mailing list