Closed (fixed)
Project:
Facets
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
4 May 2016 at 20:11 UTC
Updated:
7 Jul 2016 at 07:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #5
borisson_Comment #6
borisson_#5 was needed because I forgot the @group annotation.
Comment #9
borisson_Comment #12
borisson_I spent a big part of yesterday having another look at this, and I can't figure out why there the widget doesn't display links. I really don't know. I did figure out we don't need as many modules installed, changed that.
Comment #13
borisson_I figured it out! We needed to add the
url_processor_handlerprocessor to the facet for it to work. Test should be green now, woo.It looks like there's still something wrong with the js though, I'll try to figure that out now.

This also means that this test needs to be expanded.
Comment #14
borisson_I expanded the test, this is now looking quite good already. I did see that nothing happens when selecting a checkbox, attached screenshot from inside the test.
Because I don't think the code to take a screenshot should end up on the test-infrastructure, I'll paste it here if someone wants to see what happens.
Comment #15
strykaizerNeeded reroll, see attachment.
Looks like you can trigger the checkbox by using
Comment #16
borisson_Thanks @StryKaizer, that helped!
Comment #17
borisson_Spoke with some testing wizards at dev days, they suggested some improvements for this test. If the bot agrees, I think we can commit this.
Comment #19
borisson_Comment #20
lendude:)
Did you look at getting this from the href? Just curious if you tried and ran into something or if you just had enough of this and figured "green is green". Looking really nice now either way.
Comment #21
borisson_I did look at getting that from the href, but there was a problem with the encoding being different and even with url encoding both I couldn't get them to match up.