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

Fixed quadicon text links. #1188

Merged
merged 1 commit into from Apr 28, 2017

Conversation

h-kataria
Copy link
Contributor

  • Links when clicking on quadicon title text were broken when going to any grid/tile view thru relationships. i.e "Provider -> list of Hosts", "Provider -> List of VMs". Changed code to use same method to determine link that is being used to generate link for actual quadicon.
  • Fixed existing spec tests appropriately.

https://bugzilla.redhat.com/show_bug.cgi?id=1446212

@hayesr @dclarizio please review.

@martinpovolny
Copy link

@h-kataria : the travis failure seems related, pls, fix it

- Links when clicking on quadicon title text were broken when going to any grid/tile view thru relationships. i.e "Provider -> list of Hosts", "Provider -> List of VMs". Changed code to use same method to determine link that is being used to generate link for actual quadicon.
- Fixed existing spec tests appropriately.

https://bugzilla.redhat.com/show_bug.cgi?id=1446212
@miq-bot
Copy link
Member

miq-bot commented Apr 28, 2017

Checked commit h-kataria@1522c82 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. 👍

@dclarizio dclarizio merged commit 70ba36c into ManageIQ:master Apr 28, 2017
@dclarizio dclarizio added this to the Sprint 60 Ending May 8, 2017 milestone Apr 28, 2017
simaishi pushed a commit that referenced this pull request Apr 28, 2017
@simaishi
Copy link
Contributor

Fine backport details:

$ git log -1
commit 665cdc8bb432af32fbfebffb038791a4cf74cded
Author: Dan Clarizio <dclarizi@redhat.com>
Date:   Fri Apr 28 08:54:15 2017 -0700

    Merge pull request #1188 from h-kataria/quadicon_title_link_fix
    
    Fixed quadicon text links.
    (cherry picked from commit 70ba36ca1c67c26fc7c75443a54a84a9fbf70f62)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1446734

@h-kataria h-kataria deleted the quadicon_title_link_fix branch May 3, 2017 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants