[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/content-hub-dont-load-async into lp:ubuntu-docviewer-app

2016-05-09 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/content-hub-dont-load-async into lp:ubuntu-docviewer-app. Commit message: Do not initialize content-hub asynchronously, otherwise a transfer can not be finalized if the app is not already launched. This bug

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-terminal-app/device-detection into lp:ubuntu-terminal-app

2016-04-23 Thread Stefano Verzegnassi
I hope managers will decide to fix these problems with unmaintained projects. For now, let's suppose there has been a tacit approval. -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-terminal-app/device-detection/+merge/287985 Your team Ubuntu Terminal Developers is requested to review

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-terminal-app/device-detection into lp:ubuntu-terminal-app

2016-04-23 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-terminal-app/device-detection into lp:ubuntu-terminal-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~verzegnassi-stefano/ubuntu-terminal-app/device-detection/+merge/287985 --

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-terminal-app/fix-1488588 into lp:ubuntu-terminal-app

2016-04-23 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-terminal-app/fix-1488588 into lp:ubuntu-terminal-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~verzegnassi-stefano/ubuntu-terminal-app/fix-1488588/+merge/290031 -- Your team

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-terminal-app/uitk13-deprecation into lp:ubuntu-terminal-app

2016-04-23 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-terminal-app/uitk13-deprecation into lp:ubuntu-terminal-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~verzegnassi-stefano/ubuntu-terminal-app/uitk13-deprecation/+merge/287990 --

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-terminal-app/fix-1559892 into lp:ubuntu-terminal-app

2016-04-23 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-terminal-app/fix-1559892 into lp:ubuntu-terminal-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~verzegnassi-stefano/ubuntu-terminal-app/fix-1559892/+merge/290129 -- Your team

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-documents-page-keyboard into lp:ubuntu-docviewer-app

2016-04-20 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-documents-page-keyboard into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-documents-page-keyboard into lp:ubuntu-docviewer-app

2016-04-20 Thread Stefano Verzegnassi
I will mind to open a bug at ubuntu-ui-toolkit. I'm really missing the old PageHeadState behaviour too... :/ Thanks again for the review! -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/fix-documents-page-keyboard/+merge/291139 Your team Ubuntu Document Viewer Developers

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1566903 into lp:ubuntu-docviewer-app

2016-04-17 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1566903 into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/fix-1566903/+merge/291142 -- Your team

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1566903 into lp:ubuntu-docviewer-app

2016-04-14 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1566903 into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/fix-1566903/+merge/291142 -- Your team

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-documents-page-keyboard into lp:ubuntu-docviewer-app

2016-04-07 Thread Stefano Verzegnassi
UITK docs are not very clear on what to do when a Page have multiple PageHeaders. I used the clock-app code (more specifically, the alarm page) as reference, since you already completed the migration to UC 1.3 at the time. I'm not sure about the use of QML Loaders, since it would look a bit

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1545142 into lp:ubuntu-docviewer-app

2016-04-07 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1545142 into lp:ubuntu-docviewer-app. Commit message: PdfViewer - Presentation mode: use a single-tap gesture (instead of double tap) to show/hide header Requested reviews: Ubuntu Document Viewer

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-filemanager-app/fix-1554621 into lp:ubuntu-filemanager-app

2016-04-07 Thread Stefano Verzegnassi
Review: Approve LGTM, thanks! -- https://code.launchpad.net/~popey/ubuntu-filemanager-app/fix-1554621/+merge/291133 Your team Ubuntu File Manager Developers is requested to review the proposed merge of lp:~popey/ubuntu-filemanager-app/fix-1554621 into lp:ubuntu-filemanager-app. -- Mailing

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-filemanager-app/fix-1554621 into lp:ubuntu-filemanager-app

2016-04-07 Thread Stefano Verzegnassi
Review: Needs Fixing As to the colors, everything is okay except for a typo (see the diff comment). Sidebar color is not set and it's shown as white, when "#f7f7f7" (porcelain) should be used instead. Diff comments: > > === modified file 'src/app/qml/components/PlacesSidebar.qml' > ---

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-filemanager-app/fix-1554621 into lp:ubuntu-filemanager-app

2016-04-07 Thread Stefano Verzegnassi
Review: Needs Fixing I still have to have a better look at the app color palette, anyway there's surely something which needs to be fixed: $ grep -r "ubuntu-sdk" . ./manifest.json.in:"framework": "ubuntu-sdk-14.10", The app is still using an old framework version. Please update it to

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1566903 into lp:ubuntu-docviewer-app

2016-04-06 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1566903 into lp:ubuntu-docviewer-app. Commit message: Fixed wrong alignment of the peer picker in SharePage. Requested reviews: Ubuntu Document Viewer Developers (ubuntu-docviewer-dev) Related bugs

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-documents-page-keyboard into lp:ubuntu-docviewer-app

2016-04-06 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-documents-page-keyboard into lp:ubuntu-docviewer-app. Commit message: Fixed the keyboard input issues in DocumentsPage, caused by the migration to the new PageHeader component. In particular: * OSK

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/release-2.1.2 into lp:ubuntu-docviewer-app

2016-04-05 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/release-2.1.2 into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/release-2.1.2/+merge/290962 -- Your

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-terminal-app/device-detection into lp:ubuntu-terminal-app

2016-04-05 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-terminal-app/device-detection into lp:ubuntu-terminal-app has been updated. Description changed to: *** REQUIRES OTA-10 *** Added a context property which allows terminal-app to know whether a keyboard or a mouse is attached to the device.

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/release-2.1.2 into lp:ubuntu-docviewer-app

2016-04-05 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/release-2.1.2 into lp:ubuntu-docviewer-app. Commit message: * Bumped framework to 15.04.4 * Fixed broken night mode (i.e. app background is still white when the mode is enabled) * Updated changelog Requested

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-documents-page into lp:ubuntu-docviewer-app

2016-03-31 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-documents-page into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-lok-page into lp:ubuntu-docviewer-app

2016-03-30 Thread Stefano Verzegnassi
> Haha, are you enjoyed deleting this code? ;) Ahah, you can be sure of it! :) > Why did you switch it [PageHeader.flickable] off? Just for information. I was having trouble in setting the anchors for the content of the ViewerPage[1], and the LibreOffice Viewer had strange bindings that were

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk-theming-silo-50 into lp:ubuntu-docviewer-app

2016-03-30 Thread Stefano Verzegnassi
> Should you remove FIXME? Is it wrapping itself now? Yes, it wraps the text by default (it's limited to maximum one line of text though). I kept the comment because we may want to reserve two lines for the document name in future, however the visual result is not so good (i.e. text is - of

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-pdfview into lp:ubuntu-docviewer-app

2016-03-30 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-pdfview into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-pdfview/+merge/290172 --

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-textview into lp:ubuntu-docviewer-app

2016-03-27 Thread Stefano Verzegnassi
Yes, I'm moving all the pages to the new approach (there are already a few MPs for them). Thanks for the review! -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-textview/+merge/289482 Your team Ubuntu Document Viewer Developers is subscribed to branch

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-pdfview into lp:ubuntu-docviewer-app

2016-03-26 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-pdfview into lp:ubuntu-docviewer-app with lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk-theming-silo-50 as a prerequisite. Commit message: Pdf viewer: * Removed 'search' action (unused) * Fixed

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-documents-page into lp:ubuntu-docviewer-app

2016-03-26 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-documents-page into lp:ubuntu-docviewer-app. Commit message: * Use PageHeader and ScrollView in documents page * Removed width limitation of units.gu(80) from documents page * Code refactoring * Use

[Ubuntu-touch-coreapps-reviewers] [Bug 1559724] Re: [shorts] If night mode is enabled, the app background is still white

2016-03-25 Thread Stefano Verzegnassi
Hey Roman, I can think at three alternatives: 1) Set MainView.backgroundColor according to the nightMode status. MainView { backgroundColor: nightModeEnabled ? "black" : "white" } I'm almost sure you'll also have to disable auto-theming (i.e. UITK switches the app theme between Ambiance and

[Ubuntu-touch-coreapps-reviewers] [Bug 1377638] Re: [All Apps] Core apps should use the new splash screen features

2016-03-25 Thread Stefano Verzegnassi
** Changed in: ubuntu-terminal-app Status: Fix Committed => Fix Released -- You received this bug notification because you are a member of Ubuntu File Manager Developers, which is subscribed to Ubuntu File Manager App. Matching subscriptions: File Manager App Bugmail, Ubuntu RSS Reader

[Ubuntu-touch-coreapps-reviewers] [Bug 1288885] Re: App cannot be started on clean Trusty desktop

2016-03-25 Thread Stefano Verzegnassi
** Changed in: ubuntu-terminal-app Status: Fix Committed => Fix Released -- You received this bug notification because you are a member of Ubuntu File Manager Developers, which is subscribed to Ubuntu File Manager App. Matching subscriptions: File Manager App Bugmail

[Ubuntu-touch-coreapps-reviewers] [Bug 1559724] Re: [shorts] If night mode is enabled, the app background is still white

2016-03-25 Thread Stefano Verzegnassi
MainView now sets the background of the main window, instead of using a QML Rectangle. Since the BG is not a child item of MainView anymore, the "night mode" shader can't alter the color. Refs. - https://bugs.launchpad.net/ubuntu/+source/ubuntu-ui-toolkit/+bug/1439133 -

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-lok-page into lp:ubuntu-docviewer-app

2016-03-24 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-lok-page into lp:ubuntu-docviewer-app. Commit message: * WORKAROUND: make the lok-viewer header static (avoid unpredictable binding) * Use new PageHeader and ScrollView components * UI: Show an empty

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-terminal-app/fix-1488588 into lp:ubuntu-terminal-app

2016-03-24 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-terminal-app/fix-1488588 into lp:ubuntu-terminal-app. Commit message: * Expose drag mode settings to QML * Disabled drag support in ubuntu-terminal-app in order to workaround the missing support in Mir Requested reviews

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nskaggs/ubuntu-docviewer-app/restore-ap-template into lp:ubuntu-docviewer-app

2016-03-19 Thread Stefano Verzegnassi
Review: Approve Looks good to me! By the way, I just want to take the opportunity to thank you for the time you've spent on the core apps. Good luck with juju! :) -- https://code.launchpad.net/~nskaggs/ubuntu-docviewer-app/restore-ap-template/+merge/288454 Your team Ubuntu Document Viewer

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-textview into lp:ubuntu-docviewer-app

2016-03-19 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-textview into lp:ubuntu-docviewer-app. Commit message: [TextView] Switch to UITK 1.3 PageHeader component Requested reviews: Ubuntu Document Viewer Developers (ubuntu-docviewer-dev) For more details

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-details-page into lp:ubuntu-docviewer-app

2016-03-19 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-details-page into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1483218 into lp:ubuntu-docviewer-app

2016-03-19 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1483218 into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/fix-1483218/+merge/288689 -- Your team

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nskaggs/ubuntu-docviewer-app/restore-ap-template into lp:ubuntu-docviewer-app

2016-03-19 Thread Stefano Verzegnassi
The proposal to merge lp:~nskaggs/ubuntu-docviewer-app/restore-ap-template into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~nskaggs/ubuntu-docviewer-app/restore-ap-template/+merge/288454 -- Your team Ubuntu

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nskaggs/ubuntu-docviewer-app/create-cache into lp:ubuntu-docviewer-app

2016-03-15 Thread Stefano Verzegnassi
The proposal to merge lp:~nskaggs/ubuntu-docviewer-app/create-cache into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~nskaggs/ubuntu-docviewer-app/create-cache/+merge/288718 -- Your team Ubuntu Document Viewer

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nskaggs/ubuntu-docviewer-app/create-cache into lp:ubuntu-docviewer-app

2016-03-15 Thread Stefano Verzegnassi
The proposal to merge lp:~nskaggs/ubuntu-docviewer-app/create-cache into lp:ubuntu-docviewer-app has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~nskaggs/ubuntu-docviewer-app/create-cache/+merge/288718 -- Your team Ubuntu Document Viewer

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nskaggs/ubuntu-docviewer-app/create-cache into lp:ubuntu-docviewer-app

2016-03-15 Thread Stefano Verzegnassi
The proposal to merge lp:~nskaggs/ubuntu-docviewer-app/create-cache into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~nskaggs/ubuntu-docviewer-app/create-cache/+merge/288718 -- Your team Ubuntu Document Viewer

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nskaggs/ubuntu-docviewer-app/create-cache into lp:ubuntu-docviewer-app

2016-03-15 Thread Stefano Verzegnassi
Review: Approve -- https://code.launchpad.net/~nskaggs/ubuntu-docviewer-app/create-cache/+merge/288718 Your team Ubuntu Document Viewer Developers is subscribed to branch lp:ubuntu-docviewer-app. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers Post to :

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-terminal-app/uitk13-deprecation into lp:ubuntu-terminal-app

2016-03-15 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-terminal-app/uitk13-deprecation into lp:ubuntu-terminal-app. Commit message: * Use UITK PageHeader, instead of the now deprecated "Page.head" property * Use ScrollView where appropriate * Use UITK Labe

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-terminal-app/device-detection into lp:ubuntu-terminal-app

2016-03-15 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-terminal-app/device-detection into lp:ubuntu-terminal-app. Commit message: Added a context property which allows terminal-app to know whether a keyboard or a mouse is attached to the device. This is required to workaround

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/test into lp:ubuntu-docviewer-app

2016-03-10 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/test into lp:ubuntu-docviewer-app. Commit message: test Requested reviews: Ubuntu Document Viewer Developers (ubuntu-docviewer-dev) For more details, see: https://code.launchpad.net/~verzegnassi-stefano

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1483218 into lp:ubuntu-docviewer-app

2016-03-10 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1483218 into lp:ubuntu-docviewer-app. Commit message: emit DocumentModel::dataChanged() Sometimes the UI was not notified about changes in model entries (e.g. file size still equals to zero after file has

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/minor-code-styling into lp:ubuntu-docviewer-app

2016-03-09 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/minor-code-styling into lp:ubuntu-docviewer-app. Commit message: Minor code style changes Requested reviews: Ubuntu Document Viewer Developers (ubuntu-docviewer-dev) For more details, see: https

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-details-page into lp:ubuntu-docviewer-app

2016-03-09 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk13-details-page into lp:ubuntu-docviewer-app. Commit message: Use PageHeader and ScrollView in details page Requested reviews: Ubuntu Document Viewer Developers (ubuntu-docviewer-dev) For more details

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk-theming-silo-50 into lp:ubuntu-docviewer-app

2016-03-09 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk-theming-silo-50 into lp:ubuntu-docviewer-app. Commit message: * Updated copyright * Removed usage of deprecated colors (e.g. UbuntuColors.midAubergine) * Align to the latest clock-app specs (i.e. don't

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nskaggs/ubuntu-terminal-app/restore-autopilot into lp:ubuntu-terminal-app

2016-03-08 Thread Stefano Verzegnassi
The proposal to merge lp:~nskaggs/ubuntu-terminal-app/restore-autopilot into lp:ubuntu-terminal-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~nskaggs/ubuntu-terminal-app/restore-autopilot/+merge/288347 -- Your team Ubuntu Terminal

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nskaggs/ubuntu-terminal-app/restore-autopilot into lp:ubuntu-terminal-app

2016-03-08 Thread Stefano Verzegnassi
Review: Approve Ok, thanks! LGTM -- https://code.launchpad.net/~nskaggs/ubuntu-terminal-app/restore-autopilot/+merge/288347 Your team Ubuntu Terminal Developers is subscribed to branch lp:ubuntu-terminal-app. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers Post to

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nskaggs/ubuntu-terminal-app/restore-autopilot into lp:ubuntu-terminal-app

2016-03-08 Thread Stefano Verzegnassi
Review: Needs Information Tested on my PC and it works well. My only question is about Python coding style: should we still refer to the pep8 conventions? PS. One diff comment included (i.e. music-app typo). == $ pep8 ./tests ./tests/autopilot/ubuntu_terminal_app/tests/__init__.py:42:80:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-terminal-app/hide-terminal-data-on-auth into lp:ubuntu-terminal-app

2016-03-03 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-terminal-app/hide-terminal-data-on-auth into lp:ubuntu-terminal-app. Commit message: Hide terminal data until the access is granted (using a FastBlur). Requested reviews: Ubuntu Terminal Developers (ubuntu-terminal-dev

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mcintire-evan/ubuntu-terminal-app/window-font-size into lp:ubuntu-terminal-app

2016-03-03 Thread Stefano Verzegnassi
Huh, mark the comment about the 2px value as WORKAROUND, in case we'll decide in a distant future to move from the current code to a more "QML-friendly" implementation. -- https://code.launchpad.net/~mcintire-evan/ubuntu-terminal-app/window-font-size/+merge/285285 Your team Ubuntu Terminal

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mcintire-evan/ubuntu-terminal-app/window-font-size into lp:ubuntu-terminal-app

2016-03-03 Thread Stefano Verzegnassi
And here we go! I've found the reason of such strange behaviour! Diff comments: > > === modified file 'src/app/qml/ubuntu-terminal-app.qml' > --- src/app/qml/ubuntu-terminal-app.qml 2016-02-07 18:24:56 + > +++ src/app/qml/ubuntu-terminal-app.qml 2016-02-13 21:25:20 + > @@

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mcintire-evan/ubuntu-terminal-app/window-font-size into lp:ubuntu-terminal-app

2016-03-02 Thread Stefano Verzegnassi
I replied to your question. :) Diff comments: > === modified file 'src/app/main.cpp' > --- src/app/main.cpp 2016-01-25 12:37:14 + > +++ src/app/main.cpp 2016-02-13 21:25:20 + > @@ -53,7 +53,7 @@ > { > QApplication a(argc, argv); > QQuickView view; > -

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mcintire-evan/ubuntu-terminal-app/improve_readmes into lp:ubuntu-terminal-app

2016-03-02 Thread Stefano Verzegnassi
Review: Needs Fixing Overall it looks okay to me. Thanks for this merge proposal! :) I left 5 diff comments + two further notes (here below): 1) In "README.md": - I like the idea of providing some information about the application plugins. However, I'd prefer to provide some more general

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-terminal-app/add-control into lp:ubuntu-terminal-app

2016-02-28 Thread Stefano Verzegnassi
@Alan, any progress on this? Does it make sense to change the ownership of this branch, so that the team can finish the QML implementation? -- https://code.launchpad.net/~popey/ubuntu-terminal-app/add-control/+merge/282280 Your team Ubuntu Terminal Developers is subscribed to branch

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/cmake-click-cache-option into lp:ubuntu-docviewer-app

2016-02-25 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/cmake-click-cache-option into lp:ubuntu-docviewer-app. Commit message: Allow to disable .click dependencies caching through a CMake option Requested reviews: Ubuntu Document Viewer Developers (ubuntu

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-pdfview-be-visibility into lp:ubuntu-docviewer-app

2016-02-22 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-pdfview-be-visibility into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mcintire-evan/ubuntu-terminal-app/window-font-size into lp:ubuntu-terminal-app

2016-02-14 Thread Stefano Verzegnassi
Review: Needs Fixing I left two inline comments. As a summary, here's a diff with the changes I've proposed (line 73 is wrong - that "imports" is not required). The big problem with this MP is the usage of QQuickView::SizeViewToRootObject. In my opinion it creates a lot of problems (speaking

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mcintire-evan/ubuntu-terminal-app/auto-focus-auth into lp:ubuntu-terminal-app

2016-02-13 Thread Stefano Verzegnassi
Review: Approve Yes, it's more a workaround than a "rock-solid" solution, but it does the job well and don't require to change things where it's not strictly required. Below the Timer {}, I see that you left the Component.onCompleted handler. It isn't necessary anymore. Anyway, LGTM, so

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1544149 into lp:ubuntu-docviewer-app

2016-02-12 Thread Stefano Verzegnassi
Fixed! -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/fix-1544149/+merge/285641 Your team Ubuntu Document Viewer Developers is subscribed to branch lp:ubuntu-docviewer-app. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers Post to :

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mcintire-evan/ubuntu-terminal-app/splash-screen into lp:ubuntu-terminal-app

2016-02-11 Thread Stefano Verzegnassi
Review: Needs Fixing Sorry Evan for being that hateful guy, but the branch surely needs a fix. :) Line 9 of the diff should be: _X-Ubuntu-Splash-Title=Terminal Note the underscore before "X-Ubuntu-Splash-Title". That way "Terminal" can be properly translated into user's language. As for the

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/desktop-support2 into lp:ubuntu-docviewer-app

2016-02-11 Thread Stefano Verzegnassi
I think so too. To be fair, we shouldn't have exceptions for accessing to the user's folders too, because of this. Ideally the platform should provide some service that creates a database with a list of documents, and docviewer should be authorized to access only that database, by default.

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/release-2.1 into lp:ubuntu-docviewer-app

2016-02-11 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/release-2.1 into lp:ubuntu-docviewer-app with lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1544149 as a prerequisite. Commit message: * Release Ubuntu DocViewer 2.1 * Updated changelog Requested reviews

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/pdf-presentation-disable-screen-saver into lp:ubuntu-docviewer-app

2016-02-08 Thread Stefano Verzegnassi
@Alan: Now it should be okay. I've checked with a stopwatch and the screen saver is properly disabled. -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/pdf-presentation-disable-screen-saver/+merge/283717 Your team Ubuntu Document Viewer Developers is subscribed to branch

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mcintire-evan/ubuntu-terminal-app/disable-copy into lp:ubuntu-terminal-app

2016-02-07 Thread Stefano Verzegnassi
Review: Approve BTW, code looks good! Great work! -- https://code.launchpad.net/~mcintire-evan/ubuntu-terminal-app/disable-copy/+merge/285287 Your team Ubuntu Terminal Developers is requested to review the proposed merge of lp:~mcintire-evan/ubuntu-terminal-app/disable-copy into

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mcintire-evan/ubuntu-terminal-app/disable-paste into lp:ubuntu-terminal-app

2016-02-07 Thread Stefano Verzegnassi
@Niklas: Changing the behaviour of the .pot custom target (i.e. removing the "ALL" option) in CMake should be enough, assuming nothing else will be broken. That would mean that you'd have to explicitely do: mkdir build && cd dir cmake ../ make (bzr commit & bzr push) So it would be

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mcintire-evan/ubuntu-terminal-app/disable-paste into lp:ubuntu-terminal-app

2016-02-06 Thread Stefano Verzegnassi
Ooh, the magical world of Launchpad trying to merge a .pot update when the .pot has been updated in the meantime. :) P.S. Now you know how a DocViewer developer feels, haha :'D P.S. #2 We probably should prevent this from happening, by avoiding to update translations every time the app is built

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nikwen/ubuntu-terminal-app/improve-list-item-design into lp:ubuntu-terminal-app

2016-02-06 Thread Stefano Verzegnassi
Review: Approve Looks good to me. Thanks! -- https://code.launchpad.net/~nikwen/ubuntu-terminal-app/improve-list-item-design/+merge/285274 Your team Ubuntu Terminal Developers is requested to review the proposed merge of lp:~nikwen/ubuntu-terminal-app/improve-list-item-design into

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-terminal-app/ubuntu-color-scheme into lp:ubuntu-terminal-app

2016-02-06 Thread Stefano Verzegnassi
@Niklas: the current "Ubuntu" theme uses exactly the same palette of "DarkPastels". The only differences are the background color[1] and the blue color (which in the Ubuntu theme is shown as red). We could rename "Dark Pastels" as "Dark Pastels / Ubuntu (old)" (or something similar), but I

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/pdf-presentation-disable-screen-saver into lp:ubuntu-docviewer-app

2016-02-06 Thread Stefano Verzegnassi
Ok, I found some time for working on this. I forgot to commit the AppArmor permission for keeping the screen on (as always the problem exists between the chair and the keyboard :D) However, it keeps on turning the screen off... --

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mcintire-evan/ubuntu-terminal-app/disable-paste into lp:ubuntu-terminal-app

2016-02-05 Thread Stefano Verzegnassi
Review: Approve Tested on my BQ, and looks good. Nice job! :D Having a look at Niklas' review, I guess the "needs-fixing" is about the conflict with .pot, so there shouldn't be any problem in getting this branch merged. Anyway, here's some note for a further task: 1) I believe it's better to

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/lok-qml-async-imageprovider into lp:ubuntu-docviewer-app

2016-02-05 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/lok-qml-async-imageprovider into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/lok-qml-async-imageprovider into lp:ubuntu-docviewer-app

2016-02-05 Thread Stefano Verzegnassi
- So it seems. We've discussed about it some week ago, and it seems to keep everything in sync since it waits for the ImageResponse to complete its work on the returned image. - I moved the code that calls the RenderEngine to the image provider. This way it is used from the GUI thread and

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-terminal-app/ubuntu-color-scheme into lp:ubuntu-terminal-app

2016-02-05 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-terminal-app/ubuntu-color-scheme into lp:ubuntu-terminal-app. Commit message: * Fixed standard ANSI blue color in the Ubuntu color scheme (previously it was red) * Use the official Ubuntu palette (based on the UbuntuColors

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-terminal-app/ubuntu-color-scheme into lp:ubuntu-terminal-app

2016-02-05 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-terminal-app/ubuntu-color-scheme into lp:ubuntu-terminal-app has been updated. Description changed to: * Fixed standard ANSI blue color in the Ubuntu color scheme (previously it was red) * Use the official Ubuntu palette (based on the

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-terminal-app/ubuntu-color-scheme into lp:ubuntu-terminal-app

2016-02-05 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-terminal-app/ubuntu-color-scheme into lp:ubuntu-terminal-app has been updated. Description changed to: * Fixed standard ANSI blue color in the Ubuntu color scheme (previously it was red) * Use the official Ubuntu palette (based on the

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-terminal-app/ubuntu-color-scheme into lp:ubuntu-terminal-app

2016-02-05 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-terminal-app/ubuntu-color-scheme into lp:ubuntu-terminal-app has been updated. Status: Needs review => Work in progress For more details, see:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-terminal-app/ubuntu-color-scheme into lp:ubuntu-terminal-app

2016-02-05 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-terminal-app/ubuntu-color-scheme into lp:ubuntu-terminal-app. Commit message: * Fixed standard ANSI blue color in the Ubuntu color scheme (previously it was red) * Use the official Ubuntu palette (based on the UbuntuColors

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/desktop-support2 into lp:ubuntu-docviewer-app

2016-02-03 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/desktop-support2 into lp:ubuntu-docviewer-app. Commit message: Better desktop/pocket-desktop support, by detecting the number of mice and touchpads connected to the device. This uses QInputInfo, available

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/desktop-support2 into lp:ubuntu-docviewer-app

2016-02-03 Thread Stefano Verzegnassi
Review: Abstain Moved the MP to a new branch -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/desktop-support2/+merge/284901 Your team Ubuntu Document Viewer Developers is subscribed to branch lp:ubuntu-docviewer-app. -- Mailing list:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/pdf-presentation-action-lost into lp:ubuntu-docviewer-app

2016-02-03 Thread Stefano Verzegnassi
Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/pdf-presentation-action-lost into lp:ubuntu-docviewer-app. Commit message: Restored "Start presentation" action in the PDF viewer. After the merging of the branch with the new bottom-edge

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/desktop-support into lp:ubuntu-docviewer-app

2016-02-03 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/desktop-support into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Superseded For more details, see: https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/desktop-support/+merge/282639 --

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/desktop-support into lp:ubuntu-docviewer-app

2016-02-03 Thread Stefano Verzegnassi
Review: Disapprove I did some mistake when I merged trunk (.po conflicts). I will create a new branch for this and propose a new MP. -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/desktop-support/+merge/282639 Your team Ubuntu Document Viewer Developers is subscribed to

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/desktop-support2 into lp:ubuntu-docviewer-app

2016-02-03 Thread Stefano Verzegnassi
Review: Needs Fixing Ok, this requires an unconfined AppArmor template. No urge to add the missing packages to Jenkins, nor to merge this branch. I hope the bottom edge will be fixed upstream, because it's not currently usable with a mouse (unless we decide to run unconfined) --

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/pdf-presentation-action-lost into lp:ubuntu-docviewer-app

2016-02-03 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/pdf-presentation-action-lost into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/pdf-presentation-action-lost into lp:ubuntu-docviewer-app

2016-02-03 Thread Stefano Verzegnassi
Review: Approve Top-approving. This fix had to be included in the branches we already landed. -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/pdf-presentation-action-lost/+merge/284903 Your team Ubuntu Document Viewer Developers is subscribed to branch

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mcintire-evan/ubuntu-terminal-app/auto-focus-auth into lp:ubuntu-terminal-app

2016-02-03 Thread Stefano Verzegnassi
Review: Abstain I've tested it on my BQ and it works fine! Good work! However, giving a look at the code I saw there may be some complication in keeping this change working in future. Mine just wants to be a suggestion on how to prevent this from happening, but I understand there may be some

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/spreadsheet-part-selector into lp:ubuntu-docviewer-app

2016-02-02 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/spreadsheet-part-selector into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/lok-qml-async-imageprovider into lp:ubuntu-docviewer-app

2016-02-02 Thread Stefano Verzegnassi
Still building the project against OTA-4 (May, 2014)... -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/lok-qml-async-imageprovider/+merge/282878 Your team Ubuntu Document Viewer Developers is requested to review the proposed merge of

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mcintire-evan/ubuntu-terminal-app/auto-focus-auth into lp:ubuntu-terminal-app

2016-02-01 Thread Stefano Verzegnassi
The bug about the bluetooth keyboard should be easily reproducible on desktop too: just ensure to make the auth dialog visible and type some text with your USB/PS2/... keyboard. Currently the keyboard focus is owned by the terminal widget, that's the reason why it can be typed some command in

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mcintire-evan/ubuntu-terminal-app/auto-focus-auth into lp:ubuntu-terminal-app

2016-02-01 Thread Stefano Verzegnassi
Hi Evan! As long as you only need to test the TextField focus, you can force the auth dialog to appear on desktop too. In AuthenticationService.qml, line 33, you can remove the 'if' condition, so that the dialog will be always shown on any platform.

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/sgtileitem-keep-data into lp:ubuntu-docviewer-app

2016-01-29 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/sgtileitem-keep-data into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/new-empty-state into lp:ubuntu-docviewer-app

2016-01-29 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/new-empty-state into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/new-empty-state/+merge/283802 --

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/sgtileitem-keep-data into lp:ubuntu-docviewer-app

2016-01-29 Thread Stefano Verzegnassi
Anyway, this was the last issue with paiting we found in these 7 months. As this branch lands, we can say that the first part of the work on LOK plugin is completed! \o/ -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/sgtileitem-keep-data/+merge/283958 Your team Ubuntu

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/uitk-bottom-edge into lp:ubuntu-docviewer-app

2016-01-29 Thread Stefano Verzegnassi
It seems that Jenkins has run a build on revision 239, but it didn't update the review. Top-approving... -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/uitk-bottom-edge/+merge/281860 Your team Ubuntu Document Viewer Developers is subscribed to branch

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/spreadsheet-part-selector into lp:ubuntu-docviewer-app

2016-01-29 Thread Stefano Verzegnassi
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/spreadsheet-part-selector into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/spreadsheet-part-selector into lp:ubuntu-docviewer-app

2016-01-29 Thread Stefano Verzegnassi
It seems that Jenkins has run a build on revision 239, but it didn't update the review. Top-approving... -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/spreadsheet-part-selector/+merge/282845 Your team Ubuntu Document Viewer Developers is subscribed to branch

  1   2   3   4   >