Skip to content
This repository has been archived by the owner on Dec 17, 2017. It is now read-only.

Config dialogs #37

Closed
SokoloffA opened this issue Sep 2, 2011 · 5 comments
Closed

Config dialogs #37

SokoloffA opened this issue Sep 2, 2011 · 5 comments
Assignees
Labels
Milestone

Comments

@SokoloffA
Copy link
Member

Items in the pages list should be:

  • Read only - listWidget->setEditTriggers(QAbstractItemView::NoEditTriggers)
  • Not dragable - listWidget->setDragDropMode(QAbstractItemView::NoDragDrop)
  • Drawn on the entire list width - IMHO we need to ceate the ItemDelegate for it.
    Since it is used frequently, IMHO it is better to make a special class for list and make the promote the widgets on the forms.

Dialogs not have the buttons "Clear" and "Close."

Session and desktop config are always show the restart message, even though nothing was changed.

Config-mousetheme show not all themes, my KDE show 6 themes but razor only 1.

IMHO desktop files must have "OnlyShowIn=RAZOR;" line. http://standards.freedesktop.org/desktop-entry-spec/latest/ar01s05.html

@pvanek
Copy link
Member

pvanek commented Sep 7, 2011

fixed:

  • Read only - listWidget->setEditTriggers(QAbstractItemView::NoEditTriggers)
  • Not dragable - listWidget->setDragDropMode(QAbstractItemView::NoDragDrop)

fixed: IMHO desktop files must have "OnlyShowIn=RAZOR;"
fixed: Dialogs not have the buttons "Clear" and "Close."

@ghost ghost assigned pvanek Sep 7, 2011
pvanek added a commit that referenced this issue Sep 11, 2011
@pvanek
Copy link
Member

pvanek commented Sep 11, 2011

ad: Config-mousetheme show not all themes, my KDE show 6 themes but razor only 1.

can you try it from the terminal with this patch, please? https://gist.github.com/1209530

pvanek added a commit that referenced this issue Sep 12, 2011
…es but razor only 1. (mHidden attribute fixed)
@SokoloffA
Copy link
Member Author

Yes it's work fine.
But lets rename it to razor-config-mouse, in the future we will
add other mouse settings.

2011/9/11 pvanek <
reply@reply.github.com>

ad: Config-mousetheme show not all themes, my KDE show 6 themes but razor
only 1.

can you try it from the terminal with this patch, please?
https://gist.github.com/1209530

Reply to this email directly or view it on GitHub:
#37 (comment)

Best regards,
Alexander.

@pvanek
Copy link
Member

pvanek commented Sep 15, 2011

done

@pvanek
Copy link
Member

pvanek commented Sep 19, 2011

ok, from my side it all works now. isn't it?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

No branches or pull requests

2 participants