Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to edit/delete/enable/disable repositories when "Software Updates" section is loaded #122

Closed
alinefm opened this issue Jul 6, 2016 · 5 comments
Assignees
Labels

Comments

@alinefm
Copy link
Member

alinefm commented Jul 6, 2016

  1. Go to "Updates" tab under Host tab
  2. Open "Repositories" section
  3. Wait until it loads the content
  4. Select any repository
  5. The buttons to enable/disable, remove and edit will be enabled
  6. Open "Software Updates" section.
  7. Wait until it loads the content.
  8. Go back to "Repositories" section
  9. Select any repository.
  10. ONLY the remove button will be enabled.
    The enable/disable and edit buttons should be enabled too.
@pvital
Copy link
Member

pvital commented Jul 21, 2016

Hello @samhenri

Any update related to this issue?

@samhenri
Copy link
Contributor

samhenri commented Jul 21, 2016

Hi, I'm currently working on other issues that were tagged with higher priority. I noticed some bugs with wok.list.js but they're not related.

danielhb pushed a commit that referenced this issue Aug 5, 2016
Signed-off-by: peterpennings <peterpnns@gmail.com>
@danielhb danielhb closed this as completed Aug 5, 2016
@pvital
Copy link
Member

pvital commented Sep 13, 2016

This issue still happens with lcorreia@b51c6ae

@biancafc
Copy link
Member

Also having this issue with lcorreia@b51c6ae in Fedora 24.

@alinefm alinefm reopened this Sep 19, 2016
@alinefm alinefm assigned danielhb and unassigned samhenri Sep 21, 2016
@alinefm alinefm self-assigned this Oct 17, 2016
@danielhb danielhb removed their assignment Oct 17, 2016
@alinefm
Copy link
Member Author

alinefm commented Oct 17, 2016

Both sections, repositories and updates, make use of wok.list.js widget.
This widget is responsible for select the options on mouse click and it is searching the whole page for checked elements instead of doing that only on a given element/section.
I will send a patch to Wok to fix that. I also identified some minor issues on dashboard HTML which I will send a patch to fix too.

danielhb pushed a commit that referenced this issue Oct 18, 2016
The "Updates" tab was requesting information about the host to get
loaded which is not needed.
Also, there were some HTML elements without ID or with wrong reference to an ID
(in those cases the widget needs to know the parent (data-parent) and the content (href and aria-controls))
This patch also removes unused CSS.

All those issues were identified while investigating the cause of the
problem reported on #122.
danielhb pushed a commit that referenced this issue Oct 18, 2016
The "Updates" tab was requesting information about the host to get
loaded which is not needed.
Also, there were some HTML elements without ID or with wrong reference to an ID
(in those cases the widget needs to know the parent (data-parent) and the content (href and aria-controls))
This patch also removes unused CSS.

All those issues were identified while investigating the cause of the
problem reported on #122.
lcorreia pushed a commit to open-power-host-os/gingerbase that referenced this issue Oct 18, 2016
The "Updates" tab was requesting information about the host to get
loaded which is not needed.
Also, there were some HTML elements without ID or with wrong reference to an ID
(in those cases the widget needs to know the parent (data-parent) and the content (href and aria-controls))
This patch also removes unused CSS.

All those issues were identified while investigating the cause of the
problem reported on kimchi-project#122.
@alinefm alinefm closed this as completed Oct 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants