Page MenuHomePhabricator
Files F14009587

D12452.id29894.diff
No OneTemporary
Actions

D12452.id29894.diff
View Options

diff --git a/resources/celerity/map.php b/resources/celerity/map.php
--- a/resources/celerity/map.php
+++ b/resources/celerity/map.php
@@ -7,8 +7,8 @@
*/
return array(
'names' => array(
- 'core.pkg.css' => 'f46ba5fa',
- 'core.pkg.js' => 'b00140fe',
+ 'core.pkg.css' => '300b2bb0',
+ 'core.pkg.js' => '347cd64a',
'darkconsole.pkg.js' => '8ab24e01',
'differential.pkg.css' => '3500921f',
'differential.pkg.js' => 'c0506961',
@@ -132,7 +132,7 @@
'rsrc/css/phui/phui-document.css' => '94d5dcd8',
'rsrc/css/phui/phui-feed-story.css' => 'c9f3a0b5',
'rsrc/css/phui/phui-fontkit.css' => '1fa79503',
- 'rsrc/css/phui/phui-form-view.css' => 'b147d2ed',
+ 'rsrc/css/phui/phui-form-view.css' => 'afeb5ab3',
'rsrc/css/phui/phui-form.css' => 'f535f938',
'rsrc/css/phui/phui-header-view.css' => 'da4586b1',
'rsrc/css/phui/phui-icon.css' => 'bc766998',
@@ -222,7 +222,7 @@
'rsrc/externals/javelin/lib/__tests__/URI.js' => '1e45fda9',
'rsrc/externals/javelin/lib/__tests__/behavior.js' => '1ea62783',
'rsrc/externals/javelin/lib/behavior.js' => '61cbc29a',
- 'rsrc/externals/javelin/lib/control/tokenizer/Tokenizer.js' => 'dc708b7e',
+ 'rsrc/externals/javelin/lib/control/tokenizer/Tokenizer.js' => '3ff74d77',
'rsrc/externals/javelin/lib/control/typeahead/Typeahead.js' => '70baed2f',
'rsrc/externals/javelin/lib/control/typeahead/normalizer/TypeaheadNormalizer.js' => 'e6e25838',
'rsrc/externals/javelin/lib/control/typeahead/source/TypeaheadCompositeSource.js' => '503e17fd',
@@ -689,7 +689,7 @@
'javelin-scrollbar' => 'eaa5b321',
'javelin-sound' => '949c0fe5',
'javelin-stratcom' => '6c53634d',
- 'javelin-tokenizer' => 'dc708b7e',
+ 'javelin-tokenizer' => '3ff74d77',
'javelin-typeahead' => '70baed2f',
'javelin-typeahead-composite-source' => '503e17fd',
'javelin-typeahead-normalizer' => 'e6e25838',
@@ -791,7 +791,7 @@
'phui-font-icon-base-css' => '3dad2ae3',
'phui-fontkit-css' => '1fa79503',
'phui-form-css' => 'f535f938',
- 'phui-form-view-css' => 'b147d2ed',
+ 'phui-form-view-css' => 'afeb5ab3',
'phui-header-view-css' => 'da4586b1',
'phui-icon-view-css' => 'bc766998',
'phui-image-mask-css' => '5a8b09c8',
@@ -1093,6 +1093,12 @@
'javelin-dom',
'phortune-credit-card-form',
),
+ '3ff74d77' => array(
+ 'javelin-dom',
+ 'javelin-util',
+ 'javelin-stratcom',
+ 'javelin-install',
+ ),
'40a6a403' => array(
'javelin-install',
'javelin-dom',
@@ -1834,12 +1840,6 @@
'javelin-dom',
'phabricator-busy',
),
- 'dc708b7e' => array(
- 'javelin-dom',
- 'javelin-util',
- 'javelin-stratcom',
- 'javelin-install',
- ),
'de2e896f' => array(
'javelin-behavior',
'javelin-dom',
diff --git a/src/view/control/AphrontTokenizerTemplateView.php b/src/view/control/AphrontTokenizerTemplateView.php
--- a/src/view/control/AphrontTokenizerTemplateView.php
+++ b/src/view/control/AphrontTokenizerTemplateView.php
@@ -69,18 +69,14 @@
$content);
$icon = id(new PHUIIconView())
- ->setIconFont('fa-list-ul');
+ ->setIconFont('fa-search');
- // TODO: This thing is ugly and the ugliness is not intentional.
- // We have to give it text or PHUIButtonView collapses. It should likely
- // just be an icon and look more integrated into the input.
$browse = id(new PHUIButtonView())
->setTag('a')
->setIcon($icon)
- ->addSigil('tokenizer-browse')
+ ->addClass('tokenizer-browse-button')
->setColor(PHUIButtonView::GREY)
- ->setSize(PHUIButtonView::SMALL)
- ->setText(pht('Browse...'));
+ ->addSigil('tokenizer-browse');
$classes = array();
$classes[] = 'jx-tokenizer-frame';
@@ -90,29 +86,15 @@
}
$frame = javelin_tag(
- 'table',
+ 'div',
array(
'class' => implode(' ', $classes),
'sigil' => 'tokenizer-frame',
),
- phutil_tag(
- 'tr',
- array(
- ),
- array(
- phutil_tag(
- 'td',
- array(
- 'class' => 'jx-tokenizer-frame-input',
- ),
- $container),
- phutil_tag(
- 'td',
- array(
- 'class' => 'jx-tokenizer-frame-browse',
- ),
- $browse),
- )));
+ array(
+ $container,
+ $browse,
+ ));
return $frame;
}
diff --git a/webroot/rsrc/css/phui/phui-form-view.css b/webroot/rsrc/css/phui/phui-form-view.css
--- a/webroot/rsrc/css/phui/phui-form-view.css
+++ b/webroot/rsrc/css/phui/phui-form-view.css
@@ -40,6 +40,7 @@
margin-left: 20%;
margin-right: 20%;
width: 60%;
+ position: relative;
}
.device-phone .aphront-form-input,
@@ -496,3 +497,21 @@
.aphront-form-choose-table .aphront-form-choose-button-cell {
padding: 4px 8px;
}
+
+.button.tokenizer-browse-button {
+ position: absolute;
+ top: 1px;
+ right: 1px;
+ bottom: 1px;
+ border-radius: 0;
+ border-right: none;
+ border-top: none;
+ border-bottom: none;
+ padding: 0;
+ width: 30px;
+}
+
+.button.tokenizer-browse-button .phui-icon-view {
+ top: 7px;
+ left: 9px;
+}
diff --git a/webroot/rsrc/externals/javelin/lib/control/tokenizer/Tokenizer.js b/webroot/rsrc/externals/javelin/lib/control/tokenizer/Tokenizer.js
--- a/webroot/rsrc/externals/javelin/lib/control/tokenizer/Tokenizer.js
+++ b/webroot/rsrc/externals/javelin/lib/control/tokenizer/Tokenizer.js
@@ -79,7 +79,7 @@
this._tokenMap = {};
try {
- this._frame = JX.DOM.findAbove(this._orig, 'table', 'tokenizer-frame');
+ this._frame = JX.DOM.findAbove(this._orig, 'div', 'tokenizer-frame');
} catch (e) {
// Ignore, this tokenizer doesn't have a frame.
}

File Metadata

Mime Type
text/plain
Expires
Thu, Oct 31, 8:16 PM (2 w, 4 d ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
6748793
Default Alt Text
D12452.id29894.diff (5 KB)

Event Timeline

Phabricator · Built by Phacility