Tag related for topics particularly related to the user experience.
Not related to whatever visual change.
Not related to whatever obvious bug in the frontend.
Tag related for topics particularly related to the user experience.
Not related to whatever visual change.
Not related to whatever obvious bug in the frontend.
On the other hand, as the variable is called $html_details let's go for phutil_implode_html() to be super-safe? ping @roberto.urbani
See discussion and reasons in https://secure.phabricator.com/T11833
I agree, disabled users should be hidden in the global search typeahead results.
In T15736#15681, @bekay wrote:
- Get rid of the JX namespace and the the require comments - use import and export like it is done in modern js
In T15736#15726, @avivey wrote:to have a working dev server when developing js - at the moment I have to ./bin/celerity map after every change to js and css to see my changes...
Are you sure about that? I thought celerity map was only needed when adding a file (in dev mode).
also, adding this file in src/extensions/ will let you have css/js files from src/extensions/rsrc/ loaded automatically:
to have a working dev server when developing js - at the moment I have to ./bin/celerity map after every change to js and css to see my changes...
I don't know if that is part of this task, but the global typeahead search should hide all disabled users, wikis and repos, shouldn't it? This is nothing you want as a fast suggestion...
Well, my idea would be:
I'm triaging as "wishlist" for now, but realistically - I don't believe will ever reach a point where we'll try to fix this -- see https://secure.phabricator.com/T13154 for discussion.
So apparently the ferret search engine doesn't really have any sort of dynamic ranking. The ranking is entirely based on how many ngrams match the query...with one singular exception - all user results are boosted above everything else.
I'll try to figure out a architecturally-appropriate way to do this. The users do get grayed out in the typeahead search, just not on the results pages.
Partially related, from the mentioned search it's easy to recognize closed Tasks, but Users are not greyed out when Disabled.
Old upstream WONTFIXed this in https://secure.phabricator.com/T13154
BTW I like the proposed screenshot. So I also agree to: feel free to share the solution 👍
My fix would work with this css class
I did not realize you had a fix. Please share it. :)
In T15715#15174, @aklapper wrote:I guess you could get the result that you're looking for by editing the file webroot/rsrc/css/aphront/typeahead.css and removing the line white-space: nowrap; from the definition of div.jx-typeahead-results a.jx-result. I just don't know which side effects this may have in other places...
I guess you could get the result that you're looking for by editing the file webroot/rsrc/css/aphront/typeahead.css and removing the line white-space: nowrap; from the definition of div.jx-typeahead-results a.jx-result. I just don't know which side effects this may have in other places...
In D25419#14821, @valerio.bozzolan wrote:I'm just unsure between implode() or phutil_implode_html()
I'm just unsure between implode() or phutil_implode_html()
I believe I had some weird aggressive local browser cache pollution with rPaa8af1d79e8bfeb09e72d5e3b9330780e78b7aeb as I cannot reproduce the wordmark problem anymore, so: LGTM
+lots for removing the words "storage containers" - projects are very much not "storage".
A better metaphor is "labels" or "index" (like in a printed book? Is that something young people know about?).
Also for Archived Projects, and maybe some other objects.
I'm not sure if there's a generic way to do this, or if each search-engine needs to be updated manually.
I'd like to keep this boring:
Projects are hashtags are tags. You can use them for code bases, teams, or anything you need to group.
Completely agree on lowering their ranking as default
In addition, it seems only the owner can see their stuff here.
That's so weird! My instinct is of course to say that it should be completely unrelated to this patch since no CSS is touched in this one-line change... I have no idea how to debug this ☹️