deinstapel / deinstapel/cpupower

panel bar icon location

Open
#87 4 comments 0 reactions 1 assignee View on GitHub

@fin-ger is already working on this.

Since Apr 14, 2020.

good first issue info needed
Dominant language
JavaScript
Stars
556
Forks
72
PR merge metrics
No merged PRs in 30d

Description

Issue: icon for cpupower is to the far left, but I want clipboard manager on far left

bug; cpu power does not set status area position priority at all!

potential solution:
in extension.js
Main.panel.addToStatusArea('cpupower', _indicator._mainButton); => Main.panel.addToStatusArea('cpupower', _indicator._mainButton, 2);

Code of addToStatusArea for reference:

   addToStatusArea(role, indicator, position, box) {
        if (this.statusArea[role])
            throw new Error(`Extension point conflict: there is already a status indicator for role ${role}`);

        if (!(indicator instanceof PanelMenu.Button))
            throw new TypeError('Status indicator must be an instance of PanelMenu.Button');

        position = position || 0;
        let boxes = {
            left: this._leftBox,
            center: this._centerBox,
            right: this._rightBox
        };
        let boxContainer = boxes[box] || this._rightBox;
        this.statusArea[role] = indicator;
        this._addToPanelBox(role, indicator, position, boxContainer);
        return indicator;
    }

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.