[prev in list] [next in list] [prev in thread] [next in thread] 

List:       ktexteditor-devel
Subject:    Re: KTextEditor container extension
From:       Dominik Haumann <dhdev () gmx ! de>
Date:       2007-08-14 19:22:49
Message-ID: 200708142122.50446.dhdev () gmx ! de
[Download RAW message or body]

Moin,

not much to say. Any other comments?
Christoph? ;)

Dominik

> Index: containerinterface.h
> ===================================================================
[...]
> +class KTEXTEDITOR_EXPORT ContainerInterface
> +{
> +  public:
> +
> +    /** Virtual Destructor */
> +    ~ContainerInterface() {}

*virtual* ~ContainerInterface() {}

[...]
> + * To check if the kpart hosts supports the MDI container:
> + * \code
> + * Editor * editor = KTextEditor::EditorChooser::editor();
> + * ContainerInterface * iface = qobject_cast<ContainerInterface *>(
> editor ); + * if (iface) {
> + *   MdiContainer * mdiContainer = qobject_cast<MdiContainer *>(
> iface->container() ); + *   if (MdiContainer != NULL ) {
                                  ^ small m ;)

[...]
> +    /**
> +     * Closes the View \p view .
> +     *
> +     * The view is still valid when this call is made but will be
> deleted +     * shortly after.
> +     *
> +     * \return true if the removal is authorized and acted, or
> +     *     false if the container does not support view removing from
> +     *     the kpart, or
> +     */
> +    virtual bool closeView( View * view )=0;
> +
> +}; // class MdiContainer
> +
> +} // namespace KTextEditor
> +
> +Q_DECLARE_INTERFACE(KTextEditor::ContainerInterface, "org.kde.KTextEditor.ContainerInterface")

I'm not sure, but don't we need the following too?
Q_DECLARE_INTERFACE(KTextEditor::MdiContainer, "org.kde.KTextEditor.MdiContainer")
_______________________________________________
KTextEditor-Devel mailing list
KTextEditor-Devel@kde.org
https://mail.kde.org/mailman/listinfo/ktexteditor-devel
[prev in list] [next in list] [prev in thread] [next in thread]