> Panel/Indicators/client/IndicatorsPage.qml > > Please add your name to the "Authors" list in the header > > - property alias pageSource : page_loader.source > + property string pageSource : page_loader.source > > Let's take the opportunity to s/page_loader/pageLoader > > ----- > > Panel/Indicators/client/IndicatorsTree.qml > > + * Nick Dedekind <<email address hidden> > > Missing the closing ">" > > + Text { > + id: all_data > + color: "white" > + } > > > s/all_data/allData > > > + Indicators.ModelPrinter { > + id: printer > + model: menuModel > + > + onSourceChanged: page.refresh() > + } > > + function refresh() { > + all_data.text = printer.getString(); > + } > > > Couldn't we make it more QMLish by making the string a property of > ModelPrinter (and name it "text")? > So code would be simply: > > Text { > id: allData > color: "white" > text: printer.text > } > > ---------------- > > plugins/Unity/Indicators/modelprinter.cpp > > + * Copyright (C) 2012 Canonical, Ltd. > > s/2012/2013 > > + QString recurse_string(const QModelIndex& index, int level) const; > > s/recurse_string/recurseString > > +// Qt > +#include <QDebug> > > s/QDebug/QTextStream > > + stream << tabify(level) << roleNames[role] << "." << > iter.key() << ": " << iter.value().toString() << endl; > > Too long > > ------------------- > > plugins/Unity/Indicators/modelprinter.h > > + * Copyright (C) 2012 Canonical, Ltd. > > s/2012/2013
Thanks. All done.
« Back to merge proposal
> Panel/Indicator s/client/ IndicatorsPage. qml loader/ pageLoader s/client/ IndicatorsTree. qml ModelPrinter { getString( ); Unity/Indicator s/modelprinter. cpp string( const QModelIndex& index, int level) const; string/ recurseString QTextStream ).toString( ) << endl; Unity/Indicator s/modelprinter. h
>
> Please add your name to the "Authors" list in the header
>
> - property alias pageSource : page_loader.source
> + property string pageSource : page_loader.source
>
> Let's take the opportunity to s/page_
>
> -----
>
> Panel/Indicator
>
> + * Nick Dedekind <<email address hidden>
>
> Missing the closing ">"
>
> + Text {
> + id: all_data
> + color: "white"
> + }
>
>
> s/all_data/allData
>
>
> + Indicators.
> + id: printer
> + model: menuModel
> +
> + onSourceChanged: page.refresh()
> + }
>
> + function refresh() {
> + all_data.text = printer.
> + }
>
>
> Couldn't we make it more QMLish by making the string a property of
> ModelPrinter (and name it "text")?
> So code would be simply:
>
> Text {
> id: allData
> color: "white"
> text: printer.text
> }
>
> ----------------
>
> plugins/
>
> + * Copyright (C) 2012 Canonical, Ltd.
>
> s/2012/2013
>
> + QString recurse_
>
> s/recurse_
>
> +// Qt
> +#include <QDebug>
>
> s/QDebug/
>
> + stream << tabify(level) << roleNames[role] << "." <<
> iter.key() << ": " << iter.value(
>
> Too long
>
> -------------------
>
> plugins/
>
> + * Copyright (C) 2012 Canonical, Ltd.
>
> s/2012/2013
Thanks. All done.